[PATCH] D95094: [ShrinkWrap] Enable Shrinkwrapping on 32 and 64 bit AIX, and 64 bit ELF

Fri Feb 12 10:23:59 PST 2021

sfertile added inline comments.

================
Comment at: llvm/test/CodeGen/PowerPC/ppc-shrink-wrapping.ll:416
+; DISABLE-64AIX-NEXT: std 14, -[[STACK_OFFSET:[0-9]+]](1) # 8-byte Folded Spill
+; DISABLE-32AIX-NEXT: stw 14, -[[STACK_OFFSET:[0-9]+]](1) # 4-byte Folded Spill
+; DISABLE-NEXT: beq 0, {{.*}}[[ELSE_LABEL:BB[0-9_]+]]
----------------
Lets change the `DISABLE-32AIX` label to `DIABLE32` to highlight the differences are due to 32-bit vs 64-bit codegen and not Linux vs AIX.

================
Comment at: llvm/test/CodeGen/PowerPC/ppc-shrink-wrapping.ll:509
+;
+; DISABLE-LINUX: {{.*}}[[EPILOGUE_BB]]: # %if.end
 ; DISABLE: mtlr
----------------
`DISABLE-LINUX` --> `DISABLE`

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95094/new/

https://reviews.llvm.org/D95094