[PATCH] D35003: [MemoryBuiltins] Allow truncation in visitAllocaInst()
Mikael Holmén via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 5 04:01:47 PDT 2017
uabelho added a comment.
If the truncation actually truncates something (and not just changes e.g. an i32 1 to an i16 1)
then we are not in a good spot, but then I suppose we have already messed up since then we
would have something like
%foo = alloca i16 , i32 1000000
with pointers being 16b wide so the allocated temporary wouldn't fit in memory
anyway.
I don't know this code very well though so if you think there is a more reasonable fix for this
please let me know.
https://reviews.llvm.org/D35003
More information about the llvm-commits
mailing list