Re: PR65416, alloca on xtensa

On Fri, Mar 13, 2015 at 7:54 AM, Max Filippov jcmvb...@gmail.com wrote: 1. in windowed ABI stack pointer update is always split into two opcodes: add and movsp. How gcc optimization passes are supposed to know that 'movsp' is related to 'add' and that stack allocation is complete only after

Re: PR65416, alloca on xtensa

On Fri, Mar 13, 2015 at 10:04 AM, Marc Gauthier m...@cadence.com wrote: Other than the required 16-byte stack alignment, there's nothing in the ABI that requires these extra 16 bytes. Perhaps there was a bad implementation of the alloca exception handler at some point a long time ago that

Re: xtensa PR65730

On Sat, Apr 11, 2015 at 12:43 AM, Max Filippov jcmvb...@gmail.com wrote: On Sat, Apr 11, 2015 at 2:16 AM, augustine.sterl...@gmail.com augustine.sterl...@gmail.com wrote: On Fri, Apr 10, 2015 at 1:18 PM, Max Filippov jcmvb...@gmail.com wrote: How can we have a mulsi3 pattern that don't get

Re: xtensa PR65730

On Fri, Apr 10, 2015 at 12:15 PM, Max Filippov jcmvb...@gmail.com wrote: Then configuration w/o multiplication should call helper at -O0 and use shift at higher optimization levels? That is what I would expect.

Re: xtensa PR65730

On Fri, Apr 10, 2015 at 1:18 PM, Max Filippov jcmvb...@gmail.com wrote: Ok, then I see why this doesn't happen: mulsi3 pattern matching is conditional on TARGET_MUL32, so when TARGET_MUL32 ==0 and expand_simple_binop emits a call to a helper it's not considered mulsi3, it's just a call:

Re: why do we need xtensa-config.h?

On Tue, Sep 6, 2016 at 11:55 PM, Thomas Schwinge wrote: > Hi! > > Neither do I really know anything about Xtensa, nor do I have a lot of > experience in these parts of GCC back ends, but: There is a lot of background to know here. Unfortunately, I have no familiarity

Re: why do we need xtensa-config.h?

On Thu, Sep 8, 2016 at 8:14 AM, Oleksij Rempel <li...@rempel-privat.de> wrote: > Am 07.09.2016 um 18:21 schrieb augustine.sterl...@gmail.com: >> On Tue, Sep 6, 2016 at 11:55 PM, Thomas Schwinge >> <tho...@codesourcery.com> wrote: >>> Hi! >>> >>&g

Re: why do we need xtensa-config.h?

On Wed, Sep 21, 2016 at 1:59 AM, Oleksij Rempel wrote: > So, first step is done. Our firmware is using GCC 6.2. > > If i see it correctly, main problem are binutils. First of all we need > custom binutils to compile GCC assambler code with custom opcodes - it's > a horror.

Re: why do we need xtensa-config.h?

On Wed, Sep 7, 2016 at 9:21 AM, augustine.sterl...@gmail.com <augustine.sterl...@gmail.com> wrote: > Hope this helps, and I'm happy to answer more questions. Also, one technique commonly used by people who ship software for Xtensa is to write it such that it could compile for any varia

Re: backporting fixes for xtensa to stable branches

On Tue, May 30, 2017 at 3:26 PM, Max Filippov wrote: > Hi Sterling, > > for xtensa we have a number of bugfixes in the mainline that were never > backported to the stable branches. It'd be great having them in the stable > gcc releases instead of carrying the fixes in various

Re: [BUILDROBOT] xtensa fallout (was: Turn DECL_SECTION_NAME into string)

On Tue, Jun 24, 2014 at 10:20 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi! This is the xtensa fallout, see http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=272418 g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W

Re: [BUILDROBOT] xtensa fallout (was: Turn DECL_SECTION_NAME into string)

On Fri, Jul 4, 2014 at 2:11 PM, Jan-Benedict Glaw jbg...@lug-owl.de wrote: On Wed, 2014-06-25 08:55:57 -0700, augustine.sterl...@gmail.com augustine.sterl...@gmail.com wrote: On Tue, Jun 24, 2014 at 10:20 PM, Jan Hubicka hubi...@ucw.cz wrote: This is the xtensa fallout, see http

Re: [PATCH 0/2] xtensa: fix floating-point parts of machine description

On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov jcmvb...@gmail.com wrote: Hi Sterling, this series fixes two bugs in xtensa.md: HI Max, thanks for this. I don't see a patch though.

Re: [PATCH 2/2] xtensa: use pre- and postincrement FP load/store when available

On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov jcmvb...@gmail.com wrote: 2014-10-10 Max Filippov jcmvb...@gmail.com gcc/ * config/xtensa/xtensa.h (TARGET_HARD_FLOAT_POSTINC): new macro. * config/xtensa/xtensa.md (*lsiu, *ssiu): add dependency on !TARGET_HARD_FLOAT_POSTINC.

Re: [PATCH 1/2] xtensa: drop unimplemented floating point operations

On Sun, Oct 12, 2014 at 3:46 PM, Max Filippov jcmvb...@gmail.com wrote: xtensa ISA never implemented FP division, reciprocal, square root and inverse square root as single opcode. Remove patterns that can emit them. 2014-10-09 Max Filippov jcmvb...@gmail.com gcc/ *

Re: [PATCH] Add zero-overhead looping for xtensa backend

On Fri, Oct 10, 2014 at 6:59 AM, Felix Yang fei.yang0...@gmail.com wrote: Hi Sterling, I made some improvement to the patch. Two changes: 1. TARGET_LOOPS is now used as a condition of the doloop related patterns, which is more elegant. Fine. 2. As the trip count register of the

Re: [PATCH] Add zero-overhead looping for xtensa backend

want this patch applied as a first step, OK? Cheers, Felix On Tue, Oct 14, 2014 at 12:09 AM, augustine.sterl...@gmail.com augustine.sterl...@gmail.com wrote: On Fri, Oct 10, 2014 at 6:59 AM, Felix Yang fei.yang0...@gmail.com wrote: Hi Sterling, I made some improvement to the patch. Two

Re: [PATCH] Add zero-overhead looping for xtensa backend

at 12:09 AM, augustine.sterl...@gmail.com augustine.sterl...@gmail.com wrote: On Fri, Oct 10, 2014 at 6:59 AM, Felix Yang fei.yang0...@gmail.com wrote: Hi Sterling, I made some improvement to the patch. Two changes: 1. TARGET_LOOPS is now used as a condition of the doloop related

Re: [PATCH] Add zero-overhead looping for xtensa backend

On Tue, Oct 21, 2014 at 7:20 PM, Yangfei (Felix) felix.y...@huawei.com wrote: If the tripcount spill issue is not handled in the pattern, ICE may happen then. Here reload is trying to spill pseudo 173, but a memory operand is not allowed in zero_cost_loop_end pattern. And this is what I am

Re: [PATCH] Add zero-overhead looping for xtensa backend

On Thu, Oct 23, 2014 at 9:12 PM, Yangfei (Felix) felix.y...@huawei.com wrote: Here the key point is we need a general purpose register for the loop instruction. So the question to ask here is, How does this work today, without loop instructions? Somehow--even when it has been spilled--a branch

Re: [PATCH] Add zero-overhead looping for xtensa backend

I mean without your patch at all. On Thu, Oct 23, 2014 at 11:30 PM, Yangfei (Felix) felix.y...@huawei.com wrote: On Thu, Oct 23, 2014 at 9:12 PM, Yangfei (Felix) felix.y...@huawei.com wrote: Here the key point is we need a general purpose register for the loop instruction. So the

Re: [PATCH] Add zero-overhead looping for xtensa backend

On Thu, Oct 23, 2014 at 11:40 PM, Yangfei (Felix) felix.y...@huawei.com wrote: 1. The original xtensa port never generates loop instruction at all. 2. A port doesn't need to implement hwloop_pattern_reg hook if it has no zero-cost loop instruction. Is that clear? We are talking in circles.

Re: [xtensa] RFA: Use new rtl iterators in xtensa_tls_referenced_p

On Sat, Oct 25, 2014 at 2:17 AM, Richard Sandiford rdsandif...@googlemail.com wrote: This is part of a series to remove uses of for_each_rtx from the ports. Tested by making sure there were no code changes for gcc.dg, gcc.c-torture and g++.dg for xtensa-elf. OK to install? OK for Xtensa

Re: [PATCH] Add zero-overhead looping for xtensa backend

On Tue, Oct 28, 2014 at 5:22 AM, Yangfei (Felix) felix.y...@huawei.com wrote: Hi Sterling, How do you think about this issue? As c6x/bfin port handles this the same way, is it OK for the patch to be applied? Thanks. I have committed this patch as attached. I made a couple of minor

Re: [PATCH] Fix loop optimization when ZOL is not available in xtensa configuration

On Mon, Dec 29, 2014 at 3:12 PM, Max Filippov jcmvb...@gmail.com wrote: Currently building gcc for xtensa configuration with XCHAL_HAVE_LOOPS set to 0 fails with the following error: xtensa.c:3952: undefined reference to `reorg_loops(bool, hw_doloop_hooks*)' Only compile target-specific

Re: [RFC 2/2] Implement call0 ABI for xtensa

On Sat, Feb 28, 2015 at 10:14 AM, Max Filippov jcmvb...@gmail.com wrote: call0 is an ABI that doesn't use register windows. 2015-02-28 Max Filippov jcmvb...@gmail.com gcc/ * config/xtensa/constraints.md (a constraint): Include stack pointer in case of call0 ABI.

Re: [RFC 1/2] Turn RETURN_ADDR_IN_PREVIOUS_FRAME into C expression

On Sat, Feb 28, 2015 at 10:14 AM, Max Filippov jcmvb...@gmail.com wrote: This allows a target to support both windowed and non-windowed ABI. 2015-02-28 Max Filippov jcmvb...@gmail.com gcc/ * builtins.c (expand_builtin_return_addr): Add RETURN_ADDR_IN_PREVIOUS_FRAME to 'if'

Re: [PATCH] gcc/reload.c: Initialize several arrays before use them in find_reloads()

On Wed, Feb 25, 2015 at 6:39 AM, Jeff Law l...@redhat.com wrote: Done. jeff Thanks!

Re: [PATCH] xtensa: align data naturally when optimizing for size

On Sat, Feb 21, 2015 at 4:19 PM, Max Filippov jcmvb...@gmail.com wrote: gcc for xtensa always aligns data at least to a word boundary, even when it has smaller natural alignment. This results in unexpectedly high data section sizes and unreasonable amount of wasted space when linking objects

Re: [PATCH] gcc/reload.c: Initialize several arrays before use them in find_reloads()

On Tue, Feb 24, 2015 at 9:05 AM, Chen Gang S gang.c...@sunrus.com.cn wrote: - After this patch, it can continue compiling, but can we be sure that it always generates correct code for execution? After this patch, it should be no worse than it was--lucky. Someone is working on fixing the

Re: [PATCH] gcc/reload.c: Initialize several arrays before use them in find_reloads()

On Tue, Feb 24, 2015 at 2:05 PM, Max Filippov jcmvb...@gmail.com wrote: Hi, On Tue, Feb 24, 2015 at 6:54 PM, Jeff Law l...@redhat.com wrote: You can tackle them in any order you wish. However, I suspect fixing the xtensa backend may be easier. I don't have any good way to test xtensa, but

Re: [PATCH] gcc/reload.c: Initialize several arrays before use them in find_reloads()

On Tue, Feb 24, 2015 at 4:45 PM, Max Filippov jcmvb...@gmail.com wrote: Sterling, I was referring Jeff's patch, do you say that his patch is not the proper fix? No, I was thinking of Chen's patch. Jeff's patch is the right one. Jeff, your patch is OK for xtensa. Do you mind checking it in?

Re: [PATCH] Fix PR target/65730

On Tue, May 19, 2015 at 8:31 PM, Max Filippov jcmvb...@gmail.com wrote: 2015-05-20 Max Filippov jcmvb...@gmail.com gcc/ * config/xtensa/xtensa.c (init_alignment_context): Replace MULT by BITS_PER_UNIT with ASHIFT by exact_log2 (BITS_PER_UNIT). Approved, please apply.

Re: [PATCH v2] xtensa: implement trap pattern

On Tue, Jun 9, 2015 at 3:14 PM, Max Filippov jcmvb...@gmail.com wrote: gcc/ * config/xtensa/xtensa.h (TARGET_DEBUG): New definition. * config/xtensa/xtensa.md (define_attr type): New type trap. (define_insn trap): New definition. --- Changes v1-v2: - drop break.n,

Re: [PATCH 2/3] xtensa: use unwind-dw2-fde-dip instead of unwind-dw2-fde

On Mon, Aug 17, 2015 at 2:59 PM, Max Filippov jcmvb...@gmail.com wrote: This allows having exception cleanup code in binaries that don't register their unwind tables. 2015-08-18 Max Filippov jcmvb...@gmail.com libgcc/ * config/xtensa/t-windowed (LIB2ADDEH): Replace unwind-dw2-fde

Re: [PATCH 1/3] xtensa: reimplement register spilling

On Mon, Aug 17, 2015 at 2:59 PM, Max Filippov jcmvb...@gmail.com wrote: 2015-08-18 Max Filippov jcmvb...@gmail.com libgcc/ * config/xtensa/lib2funcs.S (__xtensa_libgcc_window_spill): Use CALL12 followed by series of ENTRY to spill windowed registers.

Re: [PATCH 3/3] xtensa: fix _Unwind_GetCFA

On Mon, Aug 17, 2015 at 2:59 PM, Max Filippov jcmvb...@gmail.com wrote: 2015-08-18 Max Filippov jcmvb...@gmail.com libgcc/ * config/xtensa/unwind-dw2-xtensa.c (_Unwind_GetCFA): Return context-sp instead of context-cfa. Approved.

Re: [PATCH] xtensa: add -mauto-litpools option

On Tue, Aug 11, 2015 at 6:09 PM, Max Filippov jcmvb...@gmail.com wrote: With support from assembler this option allows compiling huge functions, where single literal pool at the beginning of a function may not be reachable by L32R instructions at its end. Currently assembler --auto-litpools

Re: [PATCH 1/2] xtensa: fix TLS calls for call0 ABI

On Tue, Sep 15, 2015 at 3:00 AM, Max Filippov wrote: > 2015-09-15 Max Filippov > gcc/ > * config/xtensa/xtensa.c (xtensa_call_tls_desc): Use a10 or a2 > to pass TLS call argument, according to current ABI. > *

Re: [PATCH 2/2] xtensa: fix xtensa_fallback_frame_state for call0 ABI

On Tue, Sep 15, 2015 at 3:00 AM, Max Filippov wrote: > 2015-09-15 Max Filippov > gcc/ > * config/xtensa/xtensa.h (DWARF_ALT_FRAME_RETURN_COLUMN): New > definition. > (DWARF_FRAME_REGISTERS): Reserve space for one extra register in

Re: [PATCH] xtensa: add uclinux support

On Wed, Sep 30, 2015 at 8:23 AM, Max Filippov wrote: > 2015-09-30 Max Filippov > gcc/ > * config.gcc (xtensa*-*-uclinux*): New configuration. > * config/xtensa/uclinux.h: New file. > * config/xtensa/uclinux.opt: New file. > >

Re: [PATCH] xtensa: fix libgcc build with --text-section-literals

On Tue, Feb 16, 2016 at 4:35 PM, Max Filippov wrote: > Functions __muldf3_aux, __divdf3_aux, __mulsf3_aux and __divsf3_aux > don't start with leaf_entry, so they need explicit .literal_position, > otherwise libgcc build fails in the presence of --text-section-literals. > >

Re: [PATCH] xtensa: report stack usage

On Fri, Aug 26, 2016 at 12:02 AM, Max Filippov wrote: > This enables options -fstack-usage and -Wstack-usage. > > 2016-08-21 Max Filippov > gcc/ > * config/xtensa/xtensa.c (xtensa_expand_prologue): Update >

Re: [PATCH] xtensa: don't use unwind-dw2-fde-dip with elf targets

On Mon, Oct 17, 2016 at 4:23 PM, Max Filippov wrote: > Define LIB2ADDEH_XTENSA_UNWIND_DW2_FDE to unwind-dw2-fde.c in > xtensa/t-elf and to unwind-dw2-fde-dip.c in xtensa/t-linux and use > LIB2ADDEH_XTENSA_UNWIND_DW2_FDE in LIB2ADDEH definition. > > 2016-10-17 Max Filippov

Re: [PATCH] xtensa: add HW FPU sequences for DIV/SQRT/RECIP/RSQRT

On Fri, Oct 14, 2016 at 12:14 PM, Max Filippov wrote: > > Use new FPU instruction sequences documented in the ISA book to > implement __divsf3, __divdf3, __recipsf2, __recipdf2, __rsqrtsf2, > __rsqrtdf2 and __ieee754_sqrtf and __ieee754_sqrt. > > 2013-02-12 Ding-Kai Chen

Re: [PATCH] xtensa: Fix PR target/78118

On Mon, Oct 31, 2016 at 11:32 PM, Max Filippov wrote: > 2016-10-26 Max Filippov > gcc/ > * config/xtensa/xtensa-protos.h > (xtensa_use_return_instruction_p): New prototype. > * config/xtensa/xtensa.c (xtensa_current_frame_size, >

Re: [PATCH] xtensa: fix ICE on pr59037.c test

On Tue, Nov 1, 2016 at 12:11 PM, Max Filippov wrote: > xtensa gcc gets ICE on pr59037.c test because its xtensa_output_literal > function cannot handle integer literals of sizes other than 4 and 8, > whereas the test uses 16-byte int vector. > Split integer literal formatting

Re: [PATCH] xtensa: don't xfail gcc.c-torture/compile/20001226-1.c

On Tue, Nov 1, 2016 at 12:45 PM, Max Filippov wrote: > With jump trampolines implemented in binutils since 2.25 and enabled by > default this test no longer fails on xtensa. > > 2016-11-01 Max Filippov > gcc/testsuite/ > *

Re: [PATCH] xtensa: Fix PR target/78603

On Tue, Nov 29, 2016 at 2:08 PM, Max Filippov wrote: > 2016-11-29 Max Filippov > gcc/ > * config/xtensa/xtensa.c (hwloop_optimize): Don't emit zero > overhead loop start between a call and its CALL_ARG_LOCATION > note. Approved.

Re: [RFC 1/5] gcc: xtensa: allow XCHAL_* macros to be non-constant

On Mon, May 22, 2017 at 2:09 PM, Max Filippov 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. This whole

Re: [PATCH] gcc: xtensa: fix unused parameter warning

On Mon, May 29, 2017 at 4:11 AM, Max Filippov wrote: > 2017-05-28 Max Filippov > gcc/ > * config/xtensa/xtensa.c (xtensa_initial_elimination_offset): > Mark 'to' argument with ATTRIBUTE_UNUSED. This is ok.

Re: [PATCH] gcc: xtensa: fix fprintf format specifiers

On Mon, May 29, 2017 at 4:11 AM, Max Filippov wrote: > HOST_WIDE_INT may not be long as assumed in print_operand and > xtensa_emit_call. Use HOST_WIDE_INT_PRINT_DEC/HOST_WIDE_INT_PRINT_HEX > format strings instead of %ld/0x%lx. This fixes incorrect assembly code > generation

Re: [RFC 1/5] gcc: xtensa: allow XCHAL_* macros to be non-constant

On Mon, May 22, 2017 at 2:09 PM, Max Filippov 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

Re: [RFC 2/5] gcc: xtensa: make configuration dynamic

On Mon, May 22, 2017 at 2:09 PM, Max Filippov wrote: > Now that XCHAL_* macros don't have to be preprocessor constants add > include/xtensa-dynconfig.h that defines them as fields of a structure > returned from the xtensa_get_config function. > Define that structure and fill

Re: [PATCH] xtensa: add support for SSP

On Mon, May 8, 2017 at 3:55 PM, Max Filippov wrote: > gcc/ > 2017-03-27 Max Filippov > > * config/xtensa/xtensa-protos.h > (xtensa_initial_elimination_offset): New declaration. > * config/xtensa/xtensa.c

Re: [PATCH] xtensa: fix PR target/82181

On Mon, Sep 11, 2017 at 2:16 PM, Max Filippov wrote: > 2017-09-11 Max Filippov > gcc/ > * config/xtensa/xtensa.c (xtensa_mem_offset): Check that both > words of E_DImode object are reachable by xtensa_uimm8x4 access. Approved. Please

Re: [PATCH] gcc: xtensa: enable address sanitizer

On Mon, Dec 4, 2017 at 1:28 PM, Max Filippov wrote: > gcc/ > 2017-12-04 Max Filippov > > * config/xtensa/xtensa.c (xtensa_asan_shadow_offset): New > function. > (TARGET_ASAN_SHADOW_OFFSET): New macro definition. > *

Re: [PATCH] xtensa: fix PR target/65416

On Mon, Jun 18, 2018 at 3:10 PM, Max Filippov wrote: > gcc/ > 2018-06-17 Max Filippov > > * config/xtensa/xtensa.md (UNSPEC_FRAME_BLOCKAGE): New unspec > constant. > (allocate_stack, frame_blockage, *frame_blockage): New patterns. > This is OK.

Re: [PATCH] libgcc: xtensa: fix build with -mtext-section-literals

On Tue, Jan 30, 2018 at 8:02 PM, Max Filippov wrote: > > libgcc/ > 2018-01-31 Max Filippov > > * config/xtensa/ieee754-df.S (__adddf3_aux): Add > .literal_position directive. > * config/xtensa/ieee754-sf.S (__addsf3_aux):

Re: [PATCH v2] gcc: xtensa: fix NAND code in xtensa_expand_atomic

On Tue, Sep 4, 2018 at 9:42 AM Max Filippov wrote: > NAND is ~(a1 & a2), but xtensa_expand_atomic does ~a1 & a2. > That fixes libatomic tests atomic-op-{1,2}. > > gcc/ > 2018-09-04 Max Filippov > > * config/xtensa/xtensa.c (xtensa_expand_atomic): Reorder AND and > XOR

Re: [PATCH] libgcc: xtensa: fix NaN return from add/sub/mul/div helpers

On Tue, Jan 23, 2018 at 9:55 AM, Max Filippov wrote: > libgcc/ > 2018-01-22 Max Filippov > > * config/xtensa/ieee754-df.S (__addsf3, __subsf3, __mulsf3) > (__divsf3): Make NaN return value quiet. > * config/xtensa/ieee754-sf.S

Re: [PATCH v2] gcc: xtensa: don't force PIC for uclinux target

On Mon, Nov 5, 2018 at 11:07 AM Max Filippov wrote: > xtensa-uclinux uses bFLT executable file format that cannot relocate > fields representing offsets from data to code. C++ objects built as PIC > use offsets to encode FDE structures. As a result C++ exception handling > doesn't work correctly

Re: [PATCH] PR target/86814

On Sun, Dec 30, 2018 at 1:06 AM Max Filippov wrote: > > Xtensa architecture is not affected by speculation. > > gcc/ > 2018-12-30 Max Filippov > > * config/xtensa/xtensa.c (TARGET_HAVE_SPECULATION_SAFE_VALUE): > Define to speculation_safe_value_not_needed. Approved.

Re: [PATCH] gcc: move assemble_start_function / assemble_end_function to output_mi_thunk

On Tue, May 14, 2019 at 10:28 AM Max Filippov wrote: > > Let backends call assemble_start_function after they have generated > thunk function body so that a constant pool could be output if it is > required. This may help backends to avoid implementing custom constant > loading code specifically

Re: [PATCH] xtensa: fix PR target/90922

On Tue, Jun 18, 2019 at 2:27 PM Max Filippov wrote: > > Stack pointer adjustment code in prologue missed a case of no > callee-saved registers and a stack frame size bigger than 128 bytes. > Handle that case. > > This fixes the following gcc tests with call0 ABI: >

Re: [PATCH 30/30] Changes to xtensa

On Tue, Jun 25, 2019 at 1:41 PM wrote: > > From: Aaron Sawdey > > * config/xtensa/xtensa.md (movmemsi): Change name to cpymemsi. > --- > gcc/config/xtensa/xtensa.md | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) OK for xtensa.

Re: [PATCH] xtensa: fix PR target/91880

On Tue, Sep 24, 2019 at 5:41 PM Max Filippov wrote: > > Xtensa hwloop_optimize segfaults when zero overhead loop is about to be > inserted as the first instruction of the function. > Insert zero overhead loop instruction into new basic block before the > loop when basic block that precedes the