Create a new review here: http://reviews.llvm.org/D17622
Thanks, -Rong On Wed, Feb 24, 2016 at 9:22 PM, Sean Silva <chisophu...@gmail.com> wrote: > silvas added a comment. > > In http://reviews.llvm.org/D15829#360006, @xur wrote: > >> Here is the new patch that removes the auto detection of profile kind. >> >> In this patch, I replace the CC1 option of -fprofile-instr-use=<> with >> -fprofile-instrument={llvm-use|clang-use}. For the use compilation, the >> profile reuses the -fprofile-instrument-path= option. >> >> Again, all changes are of CC1 options. Driver options are intact. >> >> Some test are modified due to the -fprofile-instr-use= option change. >> >> A new test is added to test IR profile compilation. >> test/CodeGen/pgo-instrumentation.c > > > I meant in a new phabricator review. This one is has gone on for too long and > is convoluted and hard to follow. (lots of stray inline comments etc.) > > > http://reviews.llvm.org/D15829 > > > _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
