[PATCH] D119777: [X86] Introduce x86-cmov-converter-force-all
Amir Ayupov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 15 23:19:34 PST 2022
Amir 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) {
----------------
skan wrote:
> We should at least have a quick return for `ForceAll` here b/c there is no more CMOV.
What do you mean by "there's no more CMOV"? ForceAll would have to similarly collect all blocks, then collectCmovCandidates and iterate over AllCmovGroups. I don't see an early return option.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119777/new/
https://reviews.llvm.org/D119777
More information about the llvm-commits
mailing list