[PATCH] D131773: [AArch64] Add support for 256-bit non temporal loads
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 16 02:23:28 PDT 2022
fhahn added a comment.
In D131773#3725335 <https://reviews.llvm.org/D131773#3725335>, @dmgreen wrote:
> LGTM if @fhahn doesn't know of a reason to add the isKnownEven.
I can't think of any reason why this should be needed now and also couldn't find anything when looking at the original review. @zjaffal it would be great if you could post a follow-up patch that removes ` isKnownEven` from the checks for STNP or even better move the checks to a helper function used by both STNP and LDNP handling code.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D131773/new/
https://reviews.llvm.org/D131773
More information about the llvm-commits
mailing list