[PATCH] D42392: [AArch64] Add new target feature to fuse conditional select

[Evandro Menezes via Phabricator via llvm-commits]

evandro marked 4 inline comments as done.
evandro added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64MacroFusion.cpp:230
+      case AArch64::SUBSWrs:
+        if (AArch64InstrInfo::hasShiftedReg(*FirstMI))
+          return false;
----------------
fhahn wrote:
> No need for an if here I think, you could just return `!AArch64InstrInfo::hasShiftedReg(*FirstMI)` ?
Of course!


https://reviews.llvm.org/D42392