[llvm-commits] [llvm-gcc-4.2] r43399 - /llvm-gcc-4.2/trunk/gcc/llvm-convert.cpp

[Bill Wendling]

Hi Duncan,

>> No, this doesn't give the assert. :-(
>
> the memcpy seems to have the wrong alignment in the LLVM IR.  This  
> may or may not
> lead to an assert later I suppose.

That's what's wrong here. We're using the alignment from the source  
pointer as the alignment for the memcpy, which is wrong for the  
destination pointer.

> I meant: is the memcpy getting the wrong alignment for the same  
> reason as in your testcase.
>
I believe so. It should also be asserting, but isn't.

There's another discussion of this on LLVM-Dev "RFC: llvm-convert.cpp  
Patch". No one likes my original patch, but I just wrote another one  
that might be more palatable. :-)

-bw