[PATCH] D111362: [PowerPC] Add support for ROP protection for 32 bit.

[Amy Kwan via Phabricator via llvm-commits]

amyk added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.td:5486
+// specified address.
+let mayLoad = 1, hasSideEffects = 1, isCodeGenOnly = 1 in {
+def HASHCHK : XForm_XD6_RA5_RB5<31, 754, (outs),
----------------
Just wanted to double check that `isCodeGenOnly` is intended for these, right? 
I don't have a strong opinion either way but since they're not on the 64-bit version and since `hasSideEffects` is documented, it may not hurt to document why `isCodeGenOnly` is added here.


================
Comment at: llvm/test/CodeGen/PowerPC/ppc64-rop-protection-aix.ll:277
+;; Protected Zone. However, this restriction will be removed in an upcoming
+;; revision of the ABI.
+define dso_local zeroext i32 @spill(i32* nocapture readonly %in) #0 {
----------------
Would it be helpful to add a TODO to remove this comment when the next ABI revision is released?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111362