New option -flimit-function-alignment

Rainer Orth ro@CeBiTec.Uni-Bielefeld.DE
Thu Nov 24 22:28:00 GMT 2016


Hi Jeff,

>> gcc/
>> 	* common.opt (flimit-function-alignment): New.
>> 	* doc/invoke.texi (-flimit-function-alignment): Document.
>> 	* emit-rtl.h (struct rtl_data): Add max_insn_address field.
>> 	* final.c (shorten_branches): Set it.
>> 	* varasm.c (assemble_start_function): Limit alignment if
>> 	requested.
>>
>> gcc/testsuite/
>> 	* gcc.target/i386/align-limit.c: New test.
> OK.  Sorry for the long delay.

unfortunately, this broke Solaris 12/SPARC bootstrap with /bin/as:

/vol/gcc/src/hg/trunk/local/gcc/varasm.c: In function 'void assemble_start_function(tree, const char*)':
/vol/gcc/src/hg/trunk/local/gcc/varasm.c:1794:11: error: unused variable 'align_log' [-Werror=unused-variable]
       int align_log = align_functions_log;
           ^~~~~~~~~

The following fixes it and allowed bootstrap to complete successfully.
I'm going to commit it as obvious.

	Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University


2016-11-24  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>

	* varasm.c (assemble_start_function): Wrap align_log definition in
	ASM_OUTPUT_MAX_SKIP_ALIGN.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: va.patch
Type: text/x-patch
Size: 478 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20161124/f3e31c91/attachment.bin>


More information about the Gcc-patches mailing list