[PATCH] D78756: [SveEmitter] Add builtins for svreinterpret
Eli Friedman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 28 18:55:00 PDT 2020
efriedma added inline comments.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:7880
+ return Builder.CreateBitCast(Val, Ty);
+ }
+
----------------
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.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78756/new/
https://reviews.llvm.org/D78756
More information about the cfe-commits
mailing list