[PATCH] D39349: [X86] Make -march=i686 an alias of -march=pentiumpro

2017-10-26 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL316712: [X86] Make -march=i686 an alias of -march=pentiumpro (authored by ctopper). Changed prior to commit: https://reviews.llvm.org/D39349?vs=120503&id=120508#toc Repository: rL LLVM https://revie

[PATCH] D39349: [X86] Make -march=i686 an alias of -march=pentiumpro

2017-10-26 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. I suspect the gcc behavior changed when this bug was fixed https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59588 https://reviews.llvm.org/D39349 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.or

[PATCH] D39349: [X86] Make -march=i686 an alias of -march=pentiumpro

2017-10-26 Thread Chandler Carruth via Phabricator via cfe-commits
chandlerc accepted this revision. chandlerc added a comment. This revision is now accepted and ready to land. LGTM, nice. https://reviews.llvm.org/D39349 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/l

[PATCH] D39349: [X86] Make -march=i686 an alias of -march=pentiumpro

2017-10-26 Thread Craig Topper via Phabricator via cfe-commits
craig.topper created this revision. I think the only reason they are different is because we don't set __tune_i686__ for -march=i686 to match GCC. But GCC 4.9.0 seems to have changed this behavior and they do set it now. So I think they can aliases now. https://reviews.llvm.org/D39349 Files: