[llvm] [LLVM] treat `@llvm.ssub.sat` the same as `@llvm.aarch64.neon.sqsub` (PR #140454)

[Folkert de Vries via llvm-commits]

================
@@ -1795,6 +1795,34 @@ bool AArch64LegalizerInfo::legalizeIntrinsic(LegalizerHelper &Helper,
     MI.eraseFromParent();
     return true;
   }
+  case Intrinsic::aarch64_neon_sqadd: {
+    MachineIRBuilder MIB(MI);
+    if (MIB.getMRI()->getType(MI.getOperand(0).getReg()).isVector())
----------------
folkertdev wrote:

sure. is there a reason some branches use `    MachineIRBuilder MIB(MI);` and others use `MachineIRBuilder &MIB = Helper.MIRBuilder;`? 

(I'm assuming that all branches should then use the `MIB` that is defined at the top?)

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