[PATCH] align_value attribute in Clang

Bataev, Alexey a.bataev at hotmail.com
Wed Jul 23 19:29:33 PDT 2014


Hal,
probably you should add processing of this attribute to function 
mergeDeclAttribute() in SemaDecl.cpp or something like this.

Best regards,
Alexey Bataev
=============
Software Engineer
Intel Compiler Team

23 Июль 2014 г. 19:22:35, hfinkel at anl.gov писал:
> ----- Original Message -----
>> From: "Alexey Bataev" <a.bataev at hotmail.com>
>> To: hfinkel at anl.gov, "aaron ballman" <aaron.ballman at gmail.com>, bigcheesegs at gmail.com, richard at metafoo.co.uk, "a
>> bataev" <a.bataev at hotmail.com>
>> Cc: cfe-commits at cs.uiuc.edu
>> Sent: Wednesday, July 23, 2014 3:43:25 AM
>> Subject: Re: [PATCH] align_value attribute in Clang
>>
>> In icc this attribute is inherited.
>
> Interestingly, I made this:
>
> def AlignValue : InheritableAttr
>
> but I suppose that does not do this?
>
>   -Hal
>
>>
>> Best regards,
>> Alexey Bataev
>> =============
>> Software Engineer
>> Intel Compiler Team
>> Intel Corp.
>>
>> 23.07.2014 10:13, Hal Finkel пишет:
>>> One aspect perhaps worth some thought:
>>>
>>> typedef double * aligned_double_ptr
>>> __attribute__((align_value(64)));
>>> aligned_double_ptr x;
>>>
>>> void foo(decltype(x + 3) y, aligned_double_ptr z) {}
>>>
>>> Does y also carry the alignment assumption? As it stands in my
>>> current patch, the answer is no -- I think that is the desirable
>>> outcome, but I could be convinced otherwise ;)
>>>
>>> Thanks again,
>>> Hal
>>>
>>> ----- Original Message -----
>>>> From: "Richard Smith" <richard at metafoo.co.uk>
>>>> To: hfinkel at anl.gov, "aaron ballman" <aaron.ballman at gmail.com>,
>>>> bigcheesegs at gmail.com, richard at metafoo.co.uk
>>>> Cc: cfe-commits at cs.uiuc.edu
>>>> Sent: Tuesday, July 22, 2014 9:18:56 PM
>>>> Subject: Re: [PATCH] align_value attribute in Clang
>>>>
>>>> +Michael Spencer, he was considering proposing something similar
>>>> for
>>>> standardization.
>>>>
>>>> http://reviews.llvm.org/D4635
>>>>
>>>>
>>>>
>>
>> http://reviews.llvm.org/D4635
>>
>>
>>
>
> http://reviews.llvm.org/D4635
>
>




More information about the cfe-commits mailing list