[llvm] [AArch64][GlobalISel] Legalize G_STORE for v4s8 vector (PR #82498)

David Green via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 26 00:25:03 PST 2024


davemgreen wrote:

> Can you check if this is endian-safe?

I believe that as a bitcast is defined as a store+load, we end up with `store(oldtype); load(newtype); store(newtype)`, which should be endian safe. GISel doesn't handle BE yet though, so it probably isn't possible to test.

As far as I understand, Chuong was having a go at changing this to use bitcastIf instead of doing it in a custom method.

https://github.com/llvm/llvm-project/pull/82498


More information about the llvm-commits mailing list