[PATCH] D128048: Add a new clang option "-ftime-trace="

[Whitney Tsang via Phabricator via cfe-commits]

Whitney accepted this revision.
Whitney added inline comments.


================
Comment at: clang/tools/driver/cc1_main.cpp:265
+      SmallString<128> TracePath(Clang->getFrontendOpts().TimeTracePath);
+      if (llvm::sys::fs::is_directory(TracePath)) {
+        llvm::sys::path::append(TracePath, FileName);
----------------
No need braces for single instruction block. 
https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop-statements


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128048

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