[PATCH] D69786: [X86] Add support for -mvzeroupper and -mno-vzeroupper to match gcc.

2019-11-03 Thread Craig Topper via Phabricator via cfe-commits
craig.topper created this revision. craig.topper added reviewers: RKSimon, spatel. Herald added subscribers: cfe-commits, jfb, hiraditya. Herald added projects: clang, LLVM. craig.topper added a reviewer: davezarzycki. -mvzeroupper will force the vzeroupper insertion pass to run on CPUs that norma

[PATCH] D69786: [X86] Add support for -mvzeroupper and -mno-vzeroupper to match gcc.

2019-11-03 Thread David Zarzycki via Phabricator via cfe-commits
davezarzycki accepted this revision. davezarzycki added a comment. This revision is now accepted and ready to land. LGTM. Thanks! I'll close my differential proposal then. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69786/new/ https://reviews.llv

[PATCH] D69786: [X86] Add support for -mvzeroupper and -mno-vzeroupper to match gcc.

2019-11-03 Thread David Zarzycki via Phabricator via cfe-commits
davezarzycki added a comment. Actually, wait, what does it mean for a CPU without AVX to have `FeatureInsertVZEROUPPER`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69786/new/ https://reviews.llvm.org/D69786 __

[PATCH] D69786: [X86] Add support for -mvzeroupper and -mno-vzeroupper to match gcc.

2019-11-03 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. In D69786#1731942 , @davezarzycki wrote: > Actually, wait, what does it mean for a CPU without AVX to have > `FeatureInsertVZEROUPPER`? Means we’ll do vzeroupper insertion if you add -mavx to the command line and ymm is u

[PATCH] D69786: [X86] Add support for -mvzeroupper and -mno-vzeroupper to match gcc.

2019-11-04 Thread Craig Topper via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGb2b6a54f847f: [X86] Add support for -mvzeroupper and -mno-vzeroupper to match gcc (authored by craig.topper). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D6