[PATCH] D45619: [Time-report] (1) Use special new Clang flag 'FrontendTimesIsEnabled' instead of 'llvm::TimePassesIsEnabled' inside -ftime-report feature

Andrew V. Tischenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 23 05:54:51 PDT 2018


avt77 reopened this revision.
avt77 added a comment.
This revision is now accepted and ready to land.

It's terrible but my new test was failed again as result of commit of this patch!

///b/sanitizer-x86_64-linux-bootstrap/build/llvm/tools/clang/test/Frontend/ftime-report-template-decl.cpp:155:11: error: expected string not found in input
// CHECK: Code Generation Time
//          ^

I don't understand how it's possible. The same problem raised when I committed D43578 <https://reviews.llvm.org/D43578>. Obviously, there is a situation when this  test work w/o Code Generation and as result this test is fail because code generation time is zerro. Could anyone help me? How should I change the test? The simplest way is to remove this line but I don't like this idea.


Repository:
  rL LLVM

https://reviews.llvm.org/D45619





More information about the cfe-commits mailing list