Merged in r291951. Thanks, Hans
On Fri, Jan 13, 2017 at 11:46 AM, Chandler Carruth <chandl...@gmail.com> wrote: > Also need to pull in r291881. Sorry about that. > > > On Thu, Jan 12, 2017 at 3:42 PM Hans Wennborg <h...@chromium.org> wrote: >> >> On Thu, Jan 12, 2017 at 3:01 PM, Chandler Carruth <chandl...@gmail.com> >> wrote: >> >> This isn't quite right, it should either be left as-is or dropped >> >> completely. This path was allowing people to spell "-mllvm >> >> -disable-llvm-optzns" for legacy reasons, even though the correct >> >> spelling is "-Xclang -disable-llvm-optzns". There's no need to add >> >> legacy support for "-mllvm -disable-llvm-passes", since that never >> >> worked. >> > >> > >> > Yeah, I don't think this was even really intended. >> > >> > r291850+r291853 fix this to preserve -disable-llvm-optzns exactly as it >> > was. >> > >> > I've alse followed up with Hans to make sure LLVM 4 gets this. >> >> Merged in r291860. >> >> Cheers, >> Hans _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits