anemet added a comment.

In https://reviews.llvm.org/D23284#535258, @rsmith wrote:

> I think this should start with `-fdiagnostics` to group it with other similar 
> flags. `-fdiagnostics-include-hotness-in-remarks` seems too specific to me -- 
> I could imagine generating warnings from the middle-end including hotness 
> information, not just remarks. How about `-fdiagnostics-show-hotness`? (This 
> would fit nicely with the existing `-fdiagnostics-show-*` flags.)


SGTM.  I'll update the patch.


https://reviews.llvm.org/D23284



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to