search for: r82772

Hi, Andreas Neustifter <astifter-llvm at gmx.at> writes: > Maybe you can use the already available "include/llvm/Support/Format.h"? Thanks, that simplifies the patch a lot. See the attached patch. Btw, llvm-bcanalyzer.cpp seems to also use fprintf -- does mixing it with errs() cause problems and should it be converted to use format()? best regards, Timo Lindfors

...> > Andreas Neustifter <astifter-llvm at gmx.at> writes: >> Maybe you can use the already available "include/llvm/Support/ >> Format.h"? > > Thanks, that simplifies the patch a lot. See the attached patch. It looks like something similar got applied back in r82772 on 9/25. > > Btw, llvm-bcanalyzer.cpp seems to also use fprintf -- does mixing it > with errs() cause problems and should it be converted to use format()? It would be nice to switch everything to use errs(). -Chris