sdesmalen marked an inline comment as done. sdesmalen added inline comments.
================ Comment at: clang/lib/CodeGen/CGBuiltin.cpp:7880 + return Builder.CreateBitCast(Val, Ty); + } + ---------------- efriedma wrote: > I'm vaguely suspicious this might be wrong for big-endian targets. I mean, > this isn't unreasonable, but users might be surprised if svreinterpret isn't > a no-op. For SVE the loads and stores (svld1 and svst1) are all endian safe, so no special consideration needs to be taken for big endian targets. The ACLE specifies that: > The svreinterpret functions simply reinterpret a vector of one type as a > vector of another type, without changing any of the bits. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78756/new/ https://reviews.llvm.org/D78756 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits