[PATCH] D133789: [AArch64] Disable nontemproal load for Big Endian

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 14 03:56:53 PDT 2022


fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!



================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:20596
     // Handle lowering 256 bit non temporal loads into LDNP.
-    if (LoadNode->isNonTemporal() && MemVT.getSizeInBits() == 256u &&
+    // Applies to Little Endian targets only.
+    if (LoadNode->isNonTemporal() && Subtarget->isLittleEndian() &&
----------------
I'll reword this to `Handle lowering 256 bit non temporal loads into LDNP for little-endian targets.` when committing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133789



More information about the llvm-commits mailing list