[llvm] [AArch64][PAC][GISel] Add missing clobbering info to LOADgotAUTH (PR #157433)

Daniil Kovalev via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 8 09:07:50 PDT 2025


================
@@ -2914,10 +2914,15 @@ bool AArch64InstructionSelector::select(MachineInstr &I) {
     }
 
     if (OpFlags & AArch64II::MO_GOT) {
-      I.setDesc(TII.get(MF.getInfo<AArch64FunctionInfo>()->hasELFSignedGOT()
-                            ? AArch64::LOADgotAUTH
-                            : AArch64::LOADgot));
+      bool GOTIsSigned = MF.getInfo<AArch64FunctionInfo>()->hasELFSignedGOT();
----------------
kovdan01 wrote:

Nit: maybe `IsGOTSigned` would look better (having flag identifiers starting with 'is', 'has', ... seems to be a common convention).

This is completely optional and feel free to ignore.

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


More information about the llvm-commits mailing list