[PATCH] D62115: fix a issue that clang is incompatible with gcc with -H option.

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 20 03:05:05 PDT 2019


lebedev.ri added inline comments.


================
Comment at: lib/Frontend/HeaderIncludeGen.cpp:55
+  // Simplify Filename that starts with "./"
+  if (Filename.startswith("./"));
+    Filename=Filename.substr(2);
----------------
xiangzhangllvm wrote:
> Need remove ";" ? 
This was fixed but no test was added?
Was some existing test failing previously? Which one?


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

https://reviews.llvm.org/D62115





More information about the cfe-commits mailing list