[llvm] [FastISel][AArch64] Compare Instruction Miscompilation Fix (PR #75993)

[via llvm-commits]

================
@@ -1233,7 +1233,11 @@ unsigned AArch64FastISel::emitAddSub(bool UseAdd, MVT RetVT, const Value *LHS,
       isValueAvailable(RHS)) {
     if (const auto *SI = dyn_cast<BinaryOperator>(RHS))
       if (const auto *C = dyn_cast<ConstantInt>(SI->getOperand(1)))
-        if ((SI->getOpcode() == Instruction::Shl) && (C->getZExtValue() < 4)) {
+        if ((SI->getOpcode() == Instruction::Shl) && (C->getZExtValue() < 4) &&
+            !NeedExtend) {
+          // We can only fold instructions that doesn't need extension because
----------------
brendaso1 wrote:

I didn't add `ExtendType != AArch64_AM::InvalidShiftExtend` because there is already a check before this nested-if statement in line 1232. Do you think it's still better to add the check? 

https://github.com/llvm/llvm-project/pull/75993