On Wed, Aug 26, 2026 at 06:52:23PM -0500, Peter Bergner wrote: > On 8/26/26 5:36 PM, Michael Meissner wrote: > > It would be helpful to create a second patch that adds a test case for > > pr121076. You can check the patch in now, but do add a test case in > > the future. > > > > Note, care must be taken to specifiy that the codegen is for power8 or > > power9. > > > > In particular, if you use the -mdejagnu-cpu=power8, it won't override > > the default if the compiler was configured for power10 code generation > > (i.e. using the --with-cpu=power10 configuration option). You could > > explicitly use -mcpu=power8 on the target options instead of > > -mdejagnu-cpu=power8. > > The -mdejagnu-cpu=XXX option *is* the correct option to enforce a specific > -mcpu=XXX is used for a testsuite test case. It will override any compiler > default (--with-cpu=YYY) or any user trying to force a different -mcpu=ZZZ > via make check's RUNTESTFLAGS= in a testsuite test case. It's why we added > it way back when. It uses GCC SPECs magic to remove all -mcpu= options from > the command line (explicit ones from the user and implicit ones from the > default) and them renames the -mdejagnu-cpu=XXX option to -mcpu=XXX leaving > just it to compile the test case. > > It can easily be confirmed via a "gcc -v -S -mdejagnu-cpu=XXX -mcpu=YYY ..." > compile.
You are right. Thanks. -- Michael Meissner, IBM PO Box 98, Ayer, Massachusetts, USA, 01432 email: [email protected]
