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

[Kan Shengchen via Phabricator via cfe-commits]

skan marked an inline comment as done.
skan added inline comments.


================
Comment at: lib/Frontend/HeaderIncludeGen.cpp:55
+  // Simplify Filename that starts with "./"
+  if (Filename.startswith("./"));
+    Filename=Filename.substr(2);
----------------
craig.topper wrote:
> skan wrote:
> > lebedev.ri wrote:
> > > xiangzhangllvm wrote:
> > > > Need remove ";" ? 
> > > This was fixed but no test was added?
> > > Was some existing test failing previously? Which one?
> > The test is in the file 'clang_H_opt.c'  which is included in this patch.
> The extra semicolon would have caused the body of the 'if' to execute unconditionally. Did any existing test case fail for that in your local testing? Or did you not test with that mistake?
i fixed the mistake in the updated patch.  I ran the test in 'clang_H_opt.c' alone for this patch. The extra semicolon caused the body of `if` to exeute always, which didn't cause the test to fail. 


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

https://reviews.llvm.org/D62115