[PATCH] D93099: [PowerPC] Select the D-Form load if we know its offset meets the requirement

[Jinsong Ji via Phabricator via llvm-commits]

jsji added a comment.

Although a refactor is coming, we should still avoid introducing more confusion if possible.
Please update the name and comments also. Thanks.



================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.td:498
 
 // Some r+i load/store instructions (such as LD, STD, LDU, etc.) that require
 // restricted memrix (4-aligned) constants are alignment sensitive. If these
----------------
Comments also needs update.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrInfo.td:504
 
 def aligned4load : PatFrag<(ops node:$ptr), (load node:$ptr), [{
+  return isOffsetMultipleOf(N, 4) || cast<LoadSDNode>(N)->getAlignment() >= 4;
----------------
The fix has changed the semantic of this `PatFrag`, so I think we should also rename the `PatFrag` to avoid confusion.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93099