thakis added a comment. I don't have an opinion on this patch (if you force me to have one, I'm weakly in favor), but I agree with the general sentiment. When I told people to not use mllvm and Xclang before, they told me "but if I'm not supposed to use them, why are they advertised in --help"?
thakis@thakis:~/src/llvm-build$ bin/clang --help | grep Xclang -Xclang <arg> Pass <arg> to the clang compiler thakis@thakis:~/src/llvm-build$ bin/clang --help | grep mllvm -mllvm <value> Additional arguments to forward to LLVM's option processing Which to me is a valid point! Maybe we should remove them from --help or say "internal only, don't usually use" there? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55150/new/ https://reviews.llvm.org/D55150 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits