Marty E. Plummer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35083 )
Change subject: arch/ppc64: add -mregnames to assembler directives ......................................................................
Patch Set 3:
Patch Set 3:
Can you elaborate a bit more on where / why the other style register names are being emitted and what chunk(s) of build system / compiler / etc. we are essentially forcing compatibility with?
Regarding register emission, here's an experiment.
Take https://www.ibm.com/developerworks/library/l-ppc/hello-ppc64.s straight from ibm, assemble it using the toolchain built by coreboot. No problems. Disassemble the resultant object file, and objdump will use the named registers instead of the numeric ones. Make a copy of hellp-ppc64.s and edit it to use the named registers as presented in the disassembly, and attempt to assemble that. You'll get about 14 'Error: unsupported relocation against rN' errors. Add the -mregnames arg and try again, no problem. And at least on my machine the resultant hello-ppc64.o binaries have the same md5sum.
Perhaps there is a better place to stick this `-mregnames` flag but atm this is what I've came up with.