This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFC 1/5] gcc: xtensa: allow XCHAL_* macros to be non-constant
- From: "augustine dot sterling at gmail dot com" <augustine dot sterling at gmail dot com>
- To: Max Filippov <jcmvbkbc at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "linux-xtensa at linux-xtensa dot org" <linux-xtensa at linux-xtensa dot org>
- Date: Thu, 25 May 2017 11:15:45 -0700
- Subject: Re: [RFC 1/5] gcc: xtensa: allow XCHAL_* macros to be non-constant
- Authentication-results: sourceware.org; auth=none
- References: <1495487362-18969-1-git-send-email-jcmvbkbc@gmail.com> <1495487362-18969-2-git-send-email-jcmvbkbc@gmail.com>
On Mon, May 22, 2017 at 2:09 PM, Max Filippov <jcmvbkbc@gmail.com> wrote:
> XCHAL_* macros from the xtensa-config.h are used in a number of places
> that require them to be preprocessor constants. Rewrite these places so
> that non-constant XCHAL_* definitions could be used there.
>
> 2017-05-22 Max Filippov <jcmvbkbc@gmail.com>
> gcc/
> * config/xtensa/xtensa.c (xtensa_option_override): Append
> MASK_CONST16 to target_flags in the absence of TARGET_L32R.
> (hwloop_optimize, hwloop_fail, hwloop_pattern_reg,
> xtensa_doloop_hooks): Define unconditionally.
> (xtensa_reorg_loops): Only call reorg_loops in the presence of
> TARGET_LOOPS.
> * config/xtensa/xtensa.h (TARGET_L32R): New definition.
> (TARGET_DEFAULT): Remove XCHAL_HAVE_L32R condition and account
> for it in xtensa_option_override.
> (HARD_FRAME_POINTER_IS_FRAME_POINTER,
> HARD_FRAME_POINTER_IS_ARG_POINTER): New definitions.
This is OK. Please apply.