[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