On Tue, Mar 08, 2022 at 11:28:03AM -0600, Segher Boessenkool wrote: > On Fri, Feb 11, 2022 at 12:53:07PM -0500, Michael Meissner wrote: > > Ping patch for PR target/102059 to ignore implicit -mpower8-fusion that > > prevents a function targeting power9 or power10 from inlining a function > > that > > declared it needed power8 via attribute/pragma target. > > Can we just disable any effect from this flag, instead? It should just > be implied by -mcpu=power8, and be impossible to be enabled otherwise > (or disabled!)
Yes, I can do that. We should also do the same solution for power10 fusion. What I propose is to set a regular variable with the results of the -mpower8-fusion option. This option would be true by default. Then change TARGET_POWER8_FUSION to be a macro that tests whether the current tuning CPU (not CPU we are compiling code for) to check if we are tuning for a power8. I would likely then remove any TARGET_POWER8 in places that test for TARGET_POWER8_FUSION. And similarly for TARGET_POWER10_FUSION. Note, I haven't looked at Pat's latest changes for power10 fusion. Is this acceptable? -- Michael Meissner, IBM PO Box 98, Ayer, Massachusetts, USA, 01432 email: meiss...@linux.ibm.com
