[Patch 0/6][AArch64-4.7] Fix warnings.

James Greenhalgh james.greenhalgh@arm.com
Fri Feb 1 17:43:00 GMT 2013


Hi,

This patch series fixes a number of warnings in the AArch64 port on
the aarch64-4.7-branch.

The warnings fixed are:

---
[AArch64-4.7] Fix warning - Initialise generic_tunings.

    config/aarch64/aarch64.c:129:33: warning: uninitialised const ‘generic_tunings’ is invalid in C++ [-Wc++-compat]
---
[AArch64-4.7] Fix warning - aarch64_add_constant mixed code and declarations.

    config/aarch64/aarch64.c: In function ‘aarch64_add_constant’:
    config/aarch64/aarch64.c:2249:4: warning: ISO C90 forbids mixed declarations and code [-pedantic]
---
[AArch64-4.7] Fix warning - aarch64_legitimize_reload_address passes
  the wrong type to push_reload.

    config/aarch64/aarch64.c: In function ‘aarch64_legitimize_reload_address’:
    config/aarch64/aarch64.c:3641:6: warning: enum conversion when passing argument 11 of ‘push_reload’ is invalid in C++ [-Wc++-compat]
---
[AArch64-4.7] Fix warning - aarch64_trampoline_init passes the wrong
  type to emit_library_call.

    config/aarch64/aarch64.c: In function ‘aarch64_trampoline_init’:
    config/aarch64/aarch64.c:3893:8: warning: enum conversion when passing argument 2 of ‘emit_library_call’ is invalid in C++ [-Wc++-compat]
---
[AArch64-4.7] Fix warning - Mixed code and declarations in
  aarch64_simd_const_bounds.

    config/aarch64/aarch64.c: In function ‘aarch64_simd_const_bounds’:
    config/aarch64/aarch64.c:6412:3: warning: ISO C90 forbids mixed declarations and code [-pedantic]
---
[AArch64-4.7] Backport: Fix warning in aarch64.md

    config/aarch64/aarch64.md:840: warning: source missing a mode?
---

The patch series as a whole has been regression tested against
aarch64-none-elf with no regressions.

Are these patches OK to commit to aarch64-4.7-branch?

Thanks,
James Greenhalgh

---
gcc/

2013-02-01  James Greenhalgh  <james.greenhalgh@arm.com>

	* config/aarch64/aarch64.c (generic_tunings): Initialise.

2013-02-01  James Greenhalgh  <james.greenhalgh@arm.com>

	* config/aarch64/aarch64.c
  	(aarch64_add_constant): Move declaration of 'shift' above code.

2013-02-01  James Greenhalgh  <james.greenhalgh@arm.com>

	* config/aarch64/aarch64.c
	(aarch64_legitimize_reload_address): Cast 'type' before
	passing to push_reload.

2013-02-01  James Greenhalgh  <james.greenhalgh@arm.com>

	* config/aarch64/aarch64.c
	(aarch64_trampoline_init): Pass 'LCT_NORMAL' rather than '0'
	to emit_library_call.

2013-02-01  James Greenhalgh  <james.greenhalgh@arm.com>

	* config/aarch64/aarch64.c
	(aarch64_simd_const_bounds): Move declaration of 'lane' above code.

2013-02-01  James Greenhalgh  <james.greenhalgh@arm.com>

	Backport from mainline.
	2012-12-18  James Greenhalgh  <james.greenhalgh@arm.com>

	* config/aarch64/aarch64.md (insv_imm<mode>): Add modes
	for source operands.


More information about the Gcc-patches mailing list