On Thu, Oct 15, 2015 at 9:30 PM, Uros Bizjak <ubiz...@gmail.com> wrote:

>>>> Do we support -O2 -march=lakemont with
>>>>
>>>> __attribute__((target("arch=silvermont")))
>>>
>>> Hm, no.
>>>
>>
>> Do we issue an error or silently ignore
>> __attribute__((target("arch=silvermont")))?
>> If we don't support it, should we support
>>
>> -O2 -march=silvermont
>>
>> __attribute__((target("arch=lakemont")))
>
> Actually, we have to re-initialize:
>
>   opts->x_target_flags
>     |= (TARGET_DEFAULT | TARGET_SUBTARGET_DEFAULT) & 
> ~opts_set->x_target_flags;
>
> just before TARGET_SUBTARGET{32,64}_DEFAULT processing, and it will work.

No, this won't work. The value of MASK_NO_FANCY_MATH depend on
MASK_80387setting, and once fancy math bit is set, it couldn't be
cleared for march != lakemont.

It looks just like we want to error out when lakemont is enabled with -m80387.

Uros.

Reply via email to