[Bug target/101529] [11/12 Regression] wrong code with __builtin_convertvector() and __builtin_shufflevector() at -Og caused by r11-2192

2021-11-09 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101529

--- Comment #8 from CVS Commits  ---
The trunk branch has been updated by Andrew Pinski :

https://gcc.gnu.org/g:52fa771758635d9c53cddb9116e5a66fae592230

commit r12-5078-g52fa771758635d9c53cddb9116e5a66fae592230
Author: Andrew Pinski 
Date:   Sat Nov 6 06:29:13 2021 +

aarch64: [PR101529] Fix vector shuffle insertion expansion

The function aarch64_evpc_ins would reuse the target even though
it might be the same register as the two inputs.
Instead of checking to see if we can reuse the target, just use the
original input directly.

Committed as approved after bootstrapped and tested on
aarch64-linux-gnu with no regressions.

PR target/101529

gcc/ChangeLog:

* config/aarch64/aarch64.c (aarch64_evpc_ins): Don't use target
as an input, use original one.

gcc/testsuite/ChangeLog:

* c-c++-common/torture/builtin-convertvector-2.c: New test.
* c-c++-common/torture/builtin-shufflevector-2.c: New test.

[Bug target/101529] [11/12 Regression] wrong code with __builtin_convertvector() and __builtin_shufflevector() at -Og caused by r11-2192

2021-11-06 Thread pinskia at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101529

Andrew Pinski  changed:

   What|Removed |Added

URL||https://gcc.gnu.org/piperma
   ||il/gcc-patches/2021-Novembe
   ||r/583584.html
   Keywords||patch

--- Comment #7 from Andrew Pinski  ---
Patch submitted:
https://gcc.gnu.org/pipermail/gcc-patches/2021-November/583584.html

[Bug target/101529] [11/12 Regression] wrong code with __builtin_convertvector() and __builtin_shufflevector() at -Og caused by r11-2192

2021-11-06 Thread pinskia at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101529

--- Comment #6 from Andrew Pinski  ---
Created attachment 51741
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=51741=edit
patch which is testing including testcases

Full patch which is in testing right now. Added the testcases for this bug and
the dup bug too.

[Bug target/101529] [11/12 Regression] wrong code with __builtin_convertvector() and __builtin_shufflevector() at -Og caused by r11-2192

2021-11-06 Thread pinskia at gcc dot gnu.org via Gcc-bugs 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=101529

--- Comment #5 from Andrew Pinski  ---
(In reply to Andrew Pinski from comment #4)
> Patch which I am testing:

My original patch before Dmitrij took it over, actually had the correct
behavior and the used a gen_reg_rtx. :(