I started the llvm-dev thread for the instruction location tradeoffs (removing the location from instructions to improve profiling support - possibly at the cost of other debug info consumers).
The motivation for this flag is different - for added functionality that's generally (imho/based on conversations so far) pretty well justified for profile needs (the need for function start lines to make profiles somewhat resilient to unrelated code changes, etc - as mentioned in the review/patch description). On Mon, Jan 9, 2017 at 11:51 AM Eric Christopher via Phabricator < revi...@reviews.llvm.org> wrote: > echristo added a comment. > > I don't think this ever was hashed out in the llvm-dev thread? > > > https://reviews.llvm.org/D25435 > > > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits