[PATCH] D149027: [X86] Add peephole to convert `(Cmp Op32/Op64, Imm8)` -> `(Cmp Op16/Op8, Imm8)`
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 23 14:15:23 PDT 2023
craig.topper added inline comments.
================
Comment at: llvm/lib/Target/X86/X86ISelDAGToDAG.cpp:5646
+ isa<ConstantSDNode>(N1) && !isNullConstant(N1) &&
+ hasNoSignFlagUses(SDValue(Node, 0))) {
+ const APInt &C = cast<ConstantSDNode>(N1)->getAPIntValue();
----------------
craig.topper wrote:
> Do we need to check overflow flag uses too?
Maybe carry flag too?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D149027/new/
https://reviews.llvm.org/D149027
More information about the llvm-commits
mailing list