https://bugs.llvm.org/show_bug.cgi?id=43916
Sanjay Patel <spatel+l...@rotateright.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Fixed By Commit(s)| |8e34dd941cb3
Resolution|--- |FIXED
Status|CONFIRMED |RESOLVED
--- Comment #4 from Sanjay Patel <spatel+l...@rotateright.com> ---
Crashing should be fixed with:
https://reviews.llvm.org/rG8e34dd941cb3
A couple of potential follow-ups:
1. There's an unfortunate overload of the word "simple" in LLVM that threw me
off for a second. We already have this check:
if (!Store->isSimple())
But that's about volatile/atomic. We also use "simple" for types meaning a type
"that is supported natively by some processor".
2. If performance matters for the source code in the description, that could be
a separate bug. We're transferring between scalar and vector registers, and
that seems unlikely to be optimal.
--
You are receiving this mail because:
You are on the CC list for the bug._______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
