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

[Sean Fertile via Phabricator via llvm-commits]

sfertile added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/shrink-wrap.ll:2
 ; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc64le-unknown-unknown -mcpu=pwr9 | FileCheck  %s
+; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc-ibm-aix-xcoff -mcpu=pwr4 -mattr=-altivec | FileCheck  %s -check-prefix=AIX -check-prefix=32AIX
+; RUN: llc -verify-machineinstrs < %s -mtriple=powerpc64-ibm-aix-xcoff -mcpu=pwr4 -mattr=-altivec | FileCheck  %s -check-prefix=AIX -check-prefix=64AIX
----------------
I would stick with power9 for the cpu on the AIX runsteps also, just to keep consilient with the existing runstep.


================
Comment at: llvm/test/CodeGen/PowerPC/shrink-wrap.mir:3
 # RUN:   -run-pass=shrink-wrap -o - %s | FileCheck %s
+# RUN: llc -verify-machineinstrs -mcpu=pwr4 -mtriple powerpc-ibm-aix-xcoff \
+# RUN:   -run-pass=shrink-wrap -mattr=-altivec -o - %s | FileCheck %s
----------------
ditto on using same cpu as existing runstep.


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

https://reviews.llvm.org/D95094