[PATCH] D43578: -ftime-report switch support in Clang
Richard Smith - zygoloid via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 10 08:22:09 PDT 2018
rsmith added a comment.
In https://reviews.llvm.org/D43578#1062984, @thakis wrote:
> Also, please add cfe-commits to clang changes. Since this wasn't here and the patch wasn't seen by clang folks, since ftime-report-template-decl.cppwarnings is still failing on the bots (e.g. http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/16388) and since it breaks the shared bot, maybe we should revert for now and reland when the issues are addressed?
Yes, please. This was not appropriately reviewed, breaks bots, and appears to be wrong/inappropriate in multiple ways.
Repository:
rL LLVM
https://reviews.llvm.org/D43578
More information about the cfe-commits
mailing list