[PATCH] D119777: [X86] Introduce x86-cmov-converter-force-all

[Kan Shengchen via Phabricator via llvm-commits]

skan added inline comments.


================
Comment at: llvm/lib/Target/X86/X86CmovConversion.cpp:186-195
+  if (ForceMemOperand || ForceAll) {
     CmovGroups AllCmovGroups;
     SmallVector<MachineBasicBlock *, 4> Blocks;
     for (auto &MBB : MF)
       Blocks.push_back(&MBB);
     if (collectCmovCandidates(Blocks, AllCmovGroups, /*IncludeLoads*/ true)) {
       for (auto &Group : AllCmovGroups) {
----------------
We should at least have a quick return for `ForceAll` here b/c there is no more CMOV.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119777