[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