[PATCH] D121978: [X86] Rename FeatureCMPXCHG8B/FeatureCMPXCHG16B to FeatureCX8/CX16 to match CPUID.

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 18 00:06:34 PDT 2022


skan added inline comments.


================
Comment at: llvm/lib/Target/X86/X86Subtarget.h:717-719
+    // CX16 is just the CPUID bit, the instruction also requires 64-bit mode.
+    return HasCX16 && is64Bit();
+  }
----------------
I have a little concern about the name:
1.  `CMPXCHG16B`<-> `CX16` One use abbreviation but the other not, a little confusing
2.  `useCMPXCHG16B` is not so reasonable, the function is used to check if we //could// use CX16

I'm afraid adding `useCMPXCHG16B` would make `hasCX16` useless.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121978/new/

https://reviews.llvm.org/D121978



More information about the llvm-commits mailing list