[PATCH v2] ARC: gas: Generate offset with Default CFA

Nick Clifton nickc@redhat.com
Mon Jun 27 15:54:00 GMT 2016


Hi Vineet,

> gas	* config//tc-arc.c (tc_arc_frame_initial_instructions): Use
> 	cfi_add_CFA_def_cfa to generate default CFA with offset

Approved and applied.

Note:

  +  cfi_add_CFA_def_cfa(28, 0);

You need a space between the end of the function name and the opening parenthesis.

Also - you did not test the patch by running the GAS testsuite.  If you
had done so you would have found a regression: "CFI on ARC".  I have
taken care of this for you this time...

Cheers
  Nick




More information about the Binutils mailing list