This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [committed, nvptx] Add nvptx_override_options_after_change
- From: Thomas Schwinge <thomas at codesourcery dot com>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 8 Aug 2017 10:45:30 +0200
- Subject: Re: [committed, nvptx] Add nvptx_override_options_after_change
- Authentication-results: sourceware.org; auth=none
- References: <767255ef-911c-d89c-3569-785c30354166@mentor.com> <65b84334-0d77-2507-fb65-f94c87ce8d1c@mentor.com>
Hi Tom!
On Fri, 21 Jul 2017 11:49:11 +0200, Tom de Vries <Tom_deVries@mentor.com> wrote:
> this patch adds nvptx_override_options_after_change, containing a
> workaround for PR81430.
> --- a/gcc/config/nvptx/nvptx.c
> +++ b/gcc/config/nvptx/nvptx.c
> @@ -207,6 +207,17 @@ nvptx_option_override (void)
> target_flags |= MASK_SOFT_STACK | MASK_UNIFORM_SIMT;
> }
>
> +/* Implement TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE. */
> +
> +static void
> +nvptx_override_options_after_change (void)
> +{
> + /* This is a workaround for PR81430 - nvptx acceleration compilation broken
> + because of running pass_partition_blocks. This should be dealt with in the
> + common code, not in the target. */
> + flag_reorder_blocks_and_partition = 0;
> +}
Should this be reverted now that r250852 "Apply finish_options on
DECL_FUNCTION_SPECIFIC_OPTIMIZATION for ACCEL_COMPILER" has been
committed? (I'm confirming this works fine, thanks.)
Grüße
Thomas