CGCall.cpp: Don't invoke multiple Builder.CreateBitCast() on Builder.CreateMemCpy. Or we would see sideeffect incompatibility among gcc and clang.

Douglas Gregor Thu, 10 Mar 2011 07:26:09 -0800

On Mar 10, 2011, at 1:48 AM, John McCall wrote:

> On Mar 10, 2011, at 1:04 AM, NAKAMURA Takumi wrote:
> 
>> ---
>> lib/CodeGen/CGCall.cpp |    6 ++++--
>> 1 files changed, 4 insertions(+), 2 deletions(-)
>> 
>> IIRC, gcc evaluates right to left, and clang does left to right.
> 
> 
> Nice catch;  that's bitten us before.


Should we pull this into 2.9?

        - Doug
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Re: [cfe-commits] [PATCH] lib/CodeGen/CGCall.cpp: Don't invoke multiple Builder.CreateBitCast() on Builder.CreateMemCpy. Or we would see sideeffect incompatibility among gcc and clang.

Reply via email to