This revision was automatically updated to reflect the committed changes.
Closed by commit rL318529: Fix skipping of flags in
getClangStripDependencyFileAdjuster (authored by kastiglione).
Repository:
rL LLVM
https://reviews.llvm.org/D40024
Files:
cfe/trunk/lib/Tooling/ArgumentsAdjusters.cp
kastiglione updated this revision to Diff 123131.
kastiglione added a comment.
Add tests
https://reviews.llvm.org/D40024
Files:
lib/Tooling/ArgumentsAdjusters.cpp
unittests/Tooling/ToolingTest.cpp
Index: unittests/Tooling/ToolingTest.cpp
===
alexshap added a comment.
probably a test would be useful
https://reviews.llvm.org/D40024
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kastiglione updated this revision to Diff 122829.
kastiglione added a comment.
Add a continue;
https://reviews.llvm.org/D40024
Files:
lib/Tooling/ArgumentsAdjusters.cpp
Index: lib/Tooling/ArgumentsAdjusters.cpp
===
--- lib/Tool
kastiglione created this revision.
The ArgumentsAdjuster returned from `getClangStripDependencyFileAdjuster` will
skip dependency flags, and also their associated values for those flags that
take an argument. This change corrects the handling of the `-MD` and `-MMD`
flags, which do not take an arg
