alexfh added inline comments. ================ Comment at: clang-tidy/ClangTidyOptions.h:89-93 @@ -86,1 +88,7 @@ + + /// \brief Add extra compilation arguments to the end of the list. + llvm::Optional<ArgList> ExtraArgs; + + /// \brief Add extra compilation arguments to the start of the list. + llvm::Optional<ArgList> ExtraArgsBefore; }; ---------------- djasper wrote: > Are we sure we are going to need both? Maybe for now only add the one you are > actually using (and testing)? Though it's a less frequently used option, but seems to be good for consistency (with the clang tools' -extra-arg/-extra-arg-before pair of options) and for completeness. I've extended the test to verify both options and their interaction.
http://reviews.llvm.org/D14192 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
