[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