On Mon, Mar 31, 2025 at 03:51:26PM +0200, Richard Biener wrote:
> options entry. Or whether that means ix86_valid_target_attribute_inner_p
> should have created OPT_mno_sse4 from -mno-sse4.
Or OPT_msse4_1 with value 0.
Whatever.
I think your patch with keeping OPT_mno_sse4 unmodified (it could be
even
case OPT_msse4:
if (value)
{
...
}
else
{
case OPT_mno_sse4:
}
break;
) looks reasonable.
Changing ix86_valid_target_attribute_inner_p might be even better because
OPT_msse4 is RejectNegative option, so !value for it looks weird.
Jakub
