[PATCH] D38842: [CrossTU] Fix handling of Cross Translation Unit directory path

Rafael Stahl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 26 01:54:51 PDT 2017


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

I'm gonna go ahead and approve this now, because I reported the issue. Note that I'm not a regular contributor, yet!



================
Comment at: lib/CrossTU/CrossTranslationUnit.cpp:99
       else
         llvm::sys::path::append(FilePath, FileName);
       Result[FunctionLookupName] = FilePath.str().str();
----------------
Here you could drop the if/else completely and just append.


https://reviews.llvm.org/D38842





More information about the cfe-commits mailing list