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.