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:
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?
Compatibility-wise is a bit tricky. in the binutils-gdb repository, gas/ChangeLog-9697 mentions two changes which impact -mregnames in 1996, which both predate the existance of the git repo as far as I can tell. bintutils/gdb/gas/config/tc-ppc.c includes the named registers in the initial commit 252b5132c753830d5fd56823373aed85f2a0db63, which is made slightly more robust in 7e0de605cbacbbbb2531bb70506c0843aea13111 (previously the assembler would accept 'addi %f4,%cr3,%r31`, presumably interpreting that as 'addi 4, 3, 31' without context. This was in 2017. These are the only two commits in the git history that touch that particular bit of code, so presumably any 'modern' version would work.
Ah, misread. Well, mregnames was added in commit 159e6ef88fe32c13a6a9d7485efa83ab6e47be9a, which is still a commit from 1996, so should be available in general on any 'modern' toolchain