[PATCH] D85808: [Remarks][2/2] Expand remarks hotness threshold option support in more tools

Wei Wang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 18 16:25:32 PST 2020


weiwang added a comment.

In D85808#2403587 <https://reviews.llvm.org/D85808#2403587>, @tejohnson wrote:

> Thanks for adding the Driver test. I was thinking of something to test the CompilerInvocation changes, similar to your test using opt, that ensures the option has the desired behavior when invoked via clang. Looks like there is an existing test clang/test/Frontend/optimization-remark-with-hotness.c that perhaps could be extended or leveraged?

Thanks for the suggestion. I have added a new test case for clang. The source file used is the same one in the opt test case.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85808/new/

https://reviews.llvm.org/D85808



More information about the cfe-commits mailing list