[PATCH] D80160: [Tooling] Drop leading/trailing whitespace from compile_flags.txt lines

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 19 01:03:27 PDT 2020


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

thanks for remembering this :D



================
Comment at: clang/test/Tooling/fixed-database.cpp:6
 // RUN: cp "%S/Inputs/fixed-header.h" "%t/Include/"
+// RUN: echo '# this is a comment' >> %t/compile_flags.txt
 // -I flag is relative to %t (where compile_flags is), not Src/.
----------------
Not sure what this line is about, default comment marker is `\0` for `line_itearator` and we don't seem to be changing it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80160





More information about the cfe-commits mailing list