Re: [Patch ARM Refactor Builtins 4/8] Refactor "VAR" Macros

[Ramana Radhakrishnan]

On 12/11/14 17:10, James Greenhalgh wrote:

Hi,

These macros can always be defined as a base case of VAR1 and a "recursive"
case of VAR<n-1>. At the moment, the body of VAR1 is duplicated to each
macro.

This patch makes that change.

Regression tested on arm-none-linux-gnueabihf with no issues.

OK?

Thanks,
James

---
gcc/

2014-11-12  James Greenhalgh  <james.greenha...@arm.com>

        * config/arm/arm-builtins.c (VAR1): Add a comma.
        (VAR2): Rewrite in terms of VAR1.
        (VAR3-10): Likewise.
        (arm_builtins): Remove leading comma before ARM_BUILTIN_MAX.
        * config/arm/arm_neon_builtins.def: Remove trailing commas.

OK.

Ramana