[PATCH] D157406: [SVE] Expand scalable vector ISD::BITCASTs when targeting big-endian.

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm added a comment.

In D157406#4574581 <https://reviews.llvm.org/D157406#4574581>, @dmgreen wrote:

> Have you considered the rest of big endian for SVE? My understanding was that it has been mostly ignored up to this point, and that there might be other combines that are making little endian assumptions. Anything that uses a bitcast could be wrong if it changes the element size.

During initial SVE bring up we did attempt to make the implementation either big endian clean or a user visible error. However, without any real testing I can imagine there are places where this has regressed.  I've no plans to actively pipe clean full big endian support, I just spotted the bug report and figured I'd reduce the barrier to others and perhaps that'll prompt more testing and reporting.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157406