[PING^2] Respect alignment when loading up a coerced function argument

Hal Finkel hfinkel at anl.gov
Tue May 5 15:09:45 PDT 2015


----- Original Message -----
> From: "Ulrich Weigand" <Ulrich.Weigand at de.ibm.com>
> To: "cfe commits" <cfe-commits at cs.uiuc.edu>
> Sent: Monday, May 4, 2015 8:28:51 AM
> Subject: [PING^2] Respect alignment when loading up a coerced function argument
> 
> 
> Hello,
> 
> this is the last of my recent alignment-related patches that
> was not yet reviewed:
> 
> Respect alignment when loading up a coerced function argument
> http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20150330/126694.html
> 
> 
> Any comments?  As this is a wrong codegen bug, I'd like to
> get this resolved in some way ...   Thanks!

>> This patch adds a setAlignment call in one place in CreateCoercedLoad
>> where it was missing so far.  Note that as this location, we do not
>> actually know what alignment of the source location we can rely on;
>> the callers do not pass anything to this routine. 

Can we actually fix this problem so that the correct alignment can be used?

 -Hal

>> This is already
>> an issue in other places in CreateCoercedLoad; just like those places,
>> this patch simply uses an alignment of 1, and adds a FIXME.

> 
> 
> 
> Mit freundlichen Gruessen / Best Regards
> 
> Ulrich Weigand
> 
> --
>   Dr. Ulrich Weigand | Phone: +49-7031/16-3727
>   STSM, GNU/Linux compilers and toolchain
>   IBM Deutschland Research & Development GmbH
>   Vorsitzende des Aufsichtsrats: Martina Koederitz |
>   Geschäftsführung: Dirk
> Wittkopp
>   Sitz der Gesellschaft: Böblingen | Registergericht: Amtsgericht
> Stuttgart, HRB 243294
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory




More information about the cfe-commits mailing list