On Thu, Apr 24, 2025 at 12:05:06PM +0000, Kyrylo Tkachov wrote: > >>> On 24 Apr 2025, at 12:09, Jakub Jelinek <ja...@redhat.com> wrote: > >>> > >>> On Thu, Apr 24, 2025 at 09:54:09AM +0000, Kyrylo Tkachov wrote: > >>>>> I'd have expected instead of the LTO_PARTITION_DEFAULT checks one > >>>>> should be > >>>>> testing !opts_set->x_flag_lto_partition (i.e. -flto-partition=balanced > >>>>> should be the default, but when not specified explicitly, it would > >>>>> really > >>>>> match that > >>>>> error ("%<-fipa-reorder-for-locality%> is incompatible with" > >>>>> > >>>>> > >>>>> " an explicit %qs option", "-flto-partition"); > >>>>> > >>>>> > >>>>> error wording (right now -fipa-reorder-for-locality > >>>>> -flto-partition=default > >>>>> is explicit, yet no error is emittetd). > >>>> > >>>> Yes, I think I was confused about the use of opts_set. This is overly > >>>> convoluted. > >>>> So something like this? > >>> > >>> Yes, though I'd also remove the trailing comma after > >>> LTO_PARTITION_CACHE = 5 > >> > >> Right, ok to commit this to trunk and GCC 15 branch once bootstrap is > >> successful? > > > > Yes. > > Thanks, I’ve now pushed the fix to trunk and the branch.
Actually, sorry, I'm afraid the x_flag_ipa_reorder_for_locality case should be using opts and not opts_set. Because right now I see $ ./xgcc -B ./ -o abc abc.c -flto-partition=balanced -fipa-reorder-for-locality cc1: error: ‘-fipa-reorder-for-locality’ is incompatible with an explicit ‘-flto-partition’ option $ ./xgcc -B ./ -o abc abc.c -fipa-reorder-for-locality $ ./xgcc -B ./ -o abc abc.c -flto-partition=balanced -fno-ipa-reorder-for-locality cc1: error: ‘-fipa-reorder-for-locality’ is incompatible with an explicit ‘-flto-partition’ option $ ./xgcc -B ./ -o abc abc.c -fno-ipa-reorder-for-locality $ ./xgcc -B ./ -o abc abc.c -flto-partition=balanced $ ./xgcc -B ./ -o abc abc.c The third case looks undesirable, -fno-ipa-reorder-for-locality is the default and shouldn't affect anything, whether explicit or implicit. Jakub