[LLVMdev] ConstantMerge & addressSpaces

Chris Lattner clattner at apple.com
Fri Feb 12 10:17:52 PST 2010

On Feb 12, 2010, at 3:57 AM, Arnaud Allard de Grandmaison wrote:

> Hello,
> It seems I stepped on a bug with the ConstantMerge IPO pass, regarding address spaces.
> For example, if a module contains the following:
> @a = internal constant i16 224
> @b = internal addrspace(30) constant i16 224
> Although a & b have the same initializer, i think they should not be merged. Attempting to do it anyway will trigger an assertion.
> I guess the safe and easy way is to only merge constants in the 0 address space, as the other address spaces may need a target specific behaviour. The patch for this is attached.

Nice catch, I took the opportunity to clean up the constmerge pass while fixing this:


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20100212/d34b8489/attachment.html>

More information about the llvm-dev mailing list