https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
Richard Biener changed:
What|Removed |Added
CC||rsandifo at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #22 from rguenther at suse dot de ---
> Am 15.02.2024 um 19:53 schrieb tnfchris at gcc dot gnu.org
> :
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
>
> --- Comment #21 from Tamar Christina ---
> (In reply to Richard
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #21 from Tamar Christina ---
(In reply to Richard Biener from comment #18)
> diff --git a/gcc/tree-vect-slp.cc b/gcc/tree-vect-slp.cc
> index 7cf9504398c..8deeecfd4aa 100644
> --- a/gcc/tree-vect-slp.cc
> +++ b/gcc/tree-vect-slp.cc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
Richard Biener changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #19 from GCC Commits ---
The master branch has been updated by Richard Biener :
https://gcc.gnu.org/g:b312cf21afd62b43fbc5034703e2796b0c3c416d
commit r14-9011-gb312cf21afd62b43fbc5034703e2796b0c3c416d
Author: Richard Biener
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #18 from Richard Biener ---
diff --git a/gcc/tree-vect-slp.cc b/gcc/tree-vect-slp.cc
index 7cf9504398c..8deeecfd4aa 100644
--- a/gcc/tree-vect-slp.cc
+++ b/gcc/tree-vect-slp.cc
@@ -1280,8 +1280,11 @@ vect_build_slp_tree_1 (vec_info
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #17 from Richard Biener ---
I think the following fixes it, can you verify the runtime (IL looks sane, but
it uses masked scatter stores).
diff --git a/gcc/tree-vect-loop.cc b/gcc/tree-vect-loop.cc
index 9e26b09504d..5a5865c42fc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
Richard Biener changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #15 from Tamar Christina ---
and just -O3 -march=armv8-a+sve
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #14 from Tamar Christina ---
(In reply to Richard Biener from comment #13)
> I didn't add STMT_VINFO_SLP_VECT_ONLY, I'm quite sure we can now do both SLP
> of masked loads and stores, so yes, STMT_VINFO_SLP_VECT_ONLY (when we formed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #13 from Richard Biener ---
I didn't add STMT_VINFO_SLP_VECT_ONLY, I'm quite sure we can now do both SLP of
masked loads and stores, so yes, STMT_VINFO_SLP_VECT_ONLY (when we formed
a DR group of stmts we cannot combine without SLP
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
Tamar Christina changed:
What|Removed |Added
CC||rguenth at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #11 from Andrew Pinski ---
(In reply to Andrew Pinski from comment #10)
> Note I think gcc.target/aarch64/sve/mask_struct_load_3_run.c is the runtime
> failure I mentioned.
And gcc.dg/vect/vect-cond-reduc-in-order-2-signed-zero.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #10 from Andrew Pinski ---
Note I think gcc.target/aarch64/sve/mask_struct_load_3_run.c is the runtime
failure I mentioned.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #9 from Andrew Pinski ---
Note I think GCC should be able to vectorize this loop but it goes wrong.
SVE the 7 part gets lost:
```
vect__3.12_54 = .MASK_LOAD (_48, 16B, loop_mask_52);
_32 = cond_17(D) + POLY_INT_CST [16, 16];
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #8 from Andrew Pinski ---
Created attachment 57286
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=57286=edit
Testcase that shows this is wrong code
I reduced the testcase into something which shows it is wrong code too.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
Tamar Christina changed:
What|Removed |Added
CC||tnfchris at gcc dot gnu.org
---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
Richard Biener changed:
What|Removed |Added
Keywords||needs-bisection
Priority|P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
Andrew Pinski changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever confirmed|0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
Adhemerval Zanella changed:
What|Removed |Added
Resolution|DUPLICATE |---
Status|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
Richard Biener changed:
What|Removed |Added
Resolution|--- |DUPLICATE
Target Milestone|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #3 from David Binderman ---
Reduced C code seems to be:
struct median_estimator {
long median;
long step
} median_diff_ts[];
median_estimator_update_data, median_estimator_update_diff,
median_estimator_update_median,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
--- Comment #2 from David Binderman ---
The bug first seems to appear sometime between g:93f803d53b5ccaab
and g:68f7cb6cf9e8b9f2, some 39 commits.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56
David Binderman changed:
What|Removed |Added
CC||dcb314 at hotmail dot com
---
24 matches
Mail list logo
