efriedma added inline comments.
================ Comment at: lib/Frontend/FrontendTiming.cpp:14 + +#include "llvm/Support/Timer.h" + ---------------- This should include clang/Frontend/Utils.h . ================ Comment at: test/Frontend/ftime-report-template-decl.cpp:2 +// RUN: %clang %s -S -o - -ftime-report 2>&1 | FileCheck %s +// RUN: %clang %s -S -o - -fdelayed-template-parsing -DDELAYED_TEMPLATE_PARSING -ftime-report 2>&1 | FileCheck %s + ---------------- What is this test supposed to be testing? If you're just checking that we output the timers, this doesn't need to be so complicated. We generally prefer to use %clang_cc1 for tests like this. Please use -emit-llvm instead of -S if you don't actually need assembly. https://reviews.llvm.org/D45619 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
