[cfe-commits] [PATCH] Oz optimization level sets ForceSizeOpt attribute for each function

[Quentin Colombet]

On Oct 24, 2012, at 9:25 PM, Chandler Carruth <chandlerc at google.com> wrote:

> On Wed, Oct 24, 2012 at 8:58 PM, Quentin Colombet <qcolombet at apple.com> wrote:
>> Hello,
>> 
>> Following the adding of the new ForceSizeOpt attribute in LLVM (see commit r1664220), I am making a patch for clang to set this attribute on each function when the Oz optimization level is set.
> 
> Can you fix the attribute based on the code review feedback first? I'd
> rather not start generating the IR when it's about to change.

Hi Chandler,

I am not sure to know which review you are talking about.
>From my understanding, according to Eli, it was ok, and according to Evan, it was ok too but may change in the future.
I was not making the future happening now, if it was that you meant :).

Quentin
> 
>> 
>> The idea is that optimizations will be able to adapt their behavior according to this attribute. In a second step, not given in this patch, it would be possible to make this attribute available for the end user, thus enabling code size optimization focus for specific functions.
>> The patch also contains a test case that check that the attribute is set when compiled in Oz and not set otherwise.
>> 
>> Quentin
>> 
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>