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

Chandler Carruth via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 13 11:46:27 PST 2017


Also need to pull in r291881. Sorry about that.

On Thu, Jan 12, 2017 at 3:42 PM Hans Wennborg <hans at chromium.org> wrote:

> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170113/269cd86d/attachment.html>


More information about the cfe-commits mailing list