[PATCH v2] gas: sframe: Represent .cfi_undefined RA as SFrame FDE without FREs
Indu Bhagat
indu.bhagat@oracle.com
Tue Jul 22 16:26:24 GMT 2025
On 7/22/25 7:31 AM, Jens Remus wrote:
>> Meanwhile this patch below is generally OK. The text in sframe-spec
>> will need adjustment, depending on your inputs to above. Also a nit:
>> some test asm files have tab between opcode and operand.
> Isn't that the default? At least on s390x that is what I am used to.
> The AArch64 cfi-sframe-aarch64-*.s also have a tabs. It appears the
> x86-64 cfi-sframe-x86_64-*.s don't. I can change those to use blanks.
Hmm, you are right (cfi-sframe-aarch64-4.s has tabs before the .cfi_*
and space between .cfi_* and any operand, but cfi-sframe-aarch64-1.s has
tabs between instruction opcode and operand). Looks like its not done
consistently across the files.
I would keep everything from now on as:
- first tab before insn/cfi directive
- spaces to separate if necessary the following operands
(This seems to be whats most followed in gas testsuite...)
Not sure if changing the old files to use above is worth it though.
More information about the Binutils
mailing list