r290392 - Make '-disable-llvm-optzns' an alias for '-disable-llvm-passes'.

Hans Wennborg via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 12 15:42:36 PST 2017


On Thu, Jan 12, 2017 at 3:01 PM, Chandler Carruth <chandlerc at 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


More information about the cfe-commits mailing list