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

2020-04-14 Thread Max Filippov via Gcc-patches
On Tue, Apr 14, 2020 at 4:40 PM augustine.sterl...@gmail.com
 wrote:
>
> On Mon, Apr 13, 2020 at 4:39 PM Max Filippov  wrote:
> > 2020-04-13  Max Filippov  
> > gcc/
> > * config/xtensa/xtensa.md (zero_extendhisi2, zero_extendqisi2)
> > (extendhisi2_internal): Add %v1 before the load instructions.
> >
> > gcc/testsuite/
> > * gcc.target/xtensa/pr94584.c: New test.
>
> Approved. Surprised this hasn't been caught before.

Thanks. Applied to master. I'll apply it later to gcc-8 and gcc-9 branches.
I believe that we don't have use cases in linux that rely on memw associated
with volatile memory access, so it's nice to have non-linux user with different
requirements.

-- 
Thanks.
-- Max


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

2020-04-14 Thread augustine.sterling--- via Gcc-patches
On Mon, Apr 13, 2020 at 4:39 PM Max Filippov  wrote:
> 2020-04-13  Max Filippov  
> gcc/
> * config/xtensa/xtensa.md (zero_extendhisi2, zero_extendqisi2)
> (extendhisi2_internal): Add %v1 before the load instructions.
>
> gcc/testsuite/
> * gcc.target/xtensa/pr94584.c: New test.

Approved. Surprised this hasn't been caught before.