[PATCH] D96351: [AIX] Enable the default AltiVec ABI on AIX

[Zarko Todorovski via Phabricator via llvm-commits]

ZarkoCA marked 2 inline comments as done.
ZarkoCA added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/aix-csr-vector.ll:26
+; MIR32:         fixedStack:      []
+; MIR32:         INLINEASM
+; MIR32:         BLR implicit $lr, implicit $rm
----------------
sfertile wrote:
> Ideally we would either have check-nots for v20/v26/v31 before and after the INLINEASM instruction(or APP/NO_APP). Or maybe if you check for the basic-block label then check-next the following instructions to the blr might be an equivalent way to show the same understanding.
I tried to address your comment as best as I could.  Made it check for whether we load and store to those vector registers.  In some cases it was not possible to reliably check for the vector registers since, for now, we can only check for the register number in assembly. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96351