This is an archive of the discontinued LLVM Phabricator instance.

[X86][Driver] Move all of the X86 feature flags to one spot in the Options.td file and pair them up with their negations.
ClosedPublic

Authored by craig.topper on Oct 26 2017, 1:03 PM.

Details

Summary

It looks like at one time Options.td was in alphabetical order, but that looks to have long been broken. The result is that it all the no- x86 options got separated from their other friends for no good reason.

This patch puts them all together in one place with the no- paired with its none negated version.

I've kept all the SSE and AVX/AVX512 bits together since they represent a somewhat linear progression of features. The rest I just put in alphabetical order after.

Diff Detail

Repository
rL LLVM

Event Timeline

craig.topper created this revision.Oct 26 2017, 1:03 PM
echristo accepted this revision.Oct 26 2017, 1:09 PM

LGTM. Thanks.

This revision is now accepted and ready to land.Oct 26 2017, 1:09 PM
This revision was automatically updated to reflect the committed changes.