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



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

Reply via email to