[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