In any case, when you see a test failing on bots and the fix isn't obvious, revert first to get the bots back green.
On Mon, Apr 23, 2018 at 8:54 AM, Andrew V. Tischenko via Phabricator via cfe-commits <cfe-commits@lists.llvm.org> wrote: > 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 > > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits