[PATCH] D98824: [Tooling] Handle compilation databases containing commands with double dashes

Janusz Nykiel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 19 08:23:18 PDT 2021


jnykiel updated this revision to Diff 331884.
jnykiel edited the summary of this revision.
jnykiel added a comment.

I've addressed your comments (I agree with them, essentially). After implementing the insertion of '--' for problematic file names, I had to modify some interpolation unit tests to get them to consistently pass on both Windows and Unix-like systems - I made it possible to match against a relative 'non-native' path in a test, as the absolute paths in this unit test fixture always start with a drive letter on Windows, thus making it impossible to test the '-' cases there, and always start with a '/' on Unix-like systems, making the expected string different when run on Windows or Unix-like.


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

https://reviews.llvm.org/D98824

Files:
  clang/lib/Tooling/ArgumentsAdjusters.cpp
  clang/lib/Tooling/InterpolatingCompilationDatabase.cpp
  clang/lib/Tooling/Tooling.cpp
  clang/unittests/Tooling/CompilationDatabaseTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98824.331884.patch
Type: text/x-patch
Size: 5375 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210319/2a92e9c4/attachment-0001.bin>


More information about the cfe-commits mailing list