[PATCH] D107370: [X86] Split Subtarget ISA / Security / Tuning Feature Flags Definitions. NFC

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 3 09:38:45 PDT 2021


RKSimon created this revision.
RKSimon added reviewers: craig.topper, pengfei.
Herald added subscribers: jfb, hiraditya.
RKSimon requested review of this revision.
Herald added a project: LLVM.

Our list of slow/fast tuning feature flags has become pretty extensive and is randomly interleaved with ISA and Security (Retpoline etc.) flags, not even based on when the ISAs/flags were introduced, making it tricky to locate them. Plus we started treating tuning flags separately some time ago, so this patch tries to group the flags to match.

I've left them mostly in the same order within each group - I'm happy to rearrange them further if there are specific ISA or Tuning flags that you think should be kept closer together.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D107370

Files:
  llvm/lib/Target/X86/X86.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D107370.363775.patch
Type: text/x-patch
Size: 18877 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210803/4f518245/attachment.bin>


More information about the llvm-commits mailing list