<div dir="ltr">Also need to pull in r291881. Sorry about that.<br><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 12, 2017 at 3:42 PM Hans Wennborg <<a href="mailto:hans@chromium.org">hans@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Jan 12, 2017 at 3:01 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" class="gmail_msg" target="_blank">chandlerc@gmail.com</a>> wrote:<br class="gmail_msg">
>> This isn't quite right, it should either be left as-is or dropped<br class="gmail_msg">
>> completely. This path was allowing people to spell "-mllvm<br class="gmail_msg">
>> -disable-llvm-optzns" for legacy reasons, even though the correct<br class="gmail_msg">
>> spelling is "-Xclang -disable-llvm-optzns". There's no need to add<br class="gmail_msg">
>> legacy support for "-mllvm -disable-llvm-passes", since that never<br class="gmail_msg">
>> worked.<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> Yeah, I don't think this was even really intended.<br class="gmail_msg">
><br class="gmail_msg">
> r291850+r291853 fix this to preserve -disable-llvm-optzns exactly as it was.<br class="gmail_msg">
><br class="gmail_msg">
> I've alse followed up with Hans to make sure LLVM 4 gets this.<br class="gmail_msg">
<br class="gmail_msg">
Merged in r291860.<br class="gmail_msg">
<br class="gmail_msg">
Cheers,<br class="gmail_msg">
Hans<br class="gmail_msg">
</blockquote></div></div>