[PATCH] D133765: [AArch64] Add nontemporal tests for big endian.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 13 03:44:19 PDT 2022


fhahn added a comment.

In D133765#3786105 <https://reviews.llvm.org/D133765#3786105>, @t.p.northover wrote:

> Shouldn't we fix big-endian at the same time? It's difficult to justify the benefit of adding known-broken tests.

yeah we should fix the LDNP256 lowering for BE, pre-commit the test just makes it easier to see the changes for the fix. @zjaffal could you also put up a patch fixing the 256bit LDNP lowering for BE as follow-up?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133765



More information about the llvm-commits mailing list