[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:14:15 PDT 2022
fhahn added inline comments.
================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:20598
// Handle lowering 256 bit non temporal loads into LDNP.
- if (LoadNode->isNonTemporal() && MemVT.getSizeInBits() == 256u &&
+ if (LoadNode->isNonTemporal() && Subtarget->isLittleEndian() && MemVT.getSizeInBits() == 256u &&
(MemVT.getScalarSizeInBits() == 8u ||
----------------
It looks like this line is too long. Could you run clang-format on the diff? Also, can you extend the comment to say that this applies to little-endian only?
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