[PATCH] D131469: [Clang] change default storing path of `-ftime-trace`

Whitney Tsang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 11 19:19:01 PDT 2022


Whitney added inline comments.


================
Comment at: clang/tools/driver/cc1_main.cpp:260
+    SmallString<128> TracePath(Clang->getFrontendOpts().TimeTracePath);
+    assert(!TracePath.empty());
     if (auto profilerOutput = Clang->createOutputFile(
----------------
According to llvm coding standards:
To further assist with debugging, make sure to put some kind of error message in the assertion statement (which is printed if the assertion is tripped).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131469/new/

https://reviews.llvm.org/D131469



More information about the cfe-commits mailing list