diff options
| author | Kewen Lin <linkw@gcc.gnu.org> | 2019-12-04 05:10:46 +0000 |
|---|---|---|
| committer | Kewen Lin <linkw@gcc.gnu.org> | 2019-12-04 05:10:46 +0000 |
| commit | c96828f1ec5bb54524697bd0f3fb22088fda5342 (patch) | |
| tree | 16ab3a269c12b6ff25b2645e7b20663754acd7f0 /gcc | |
| parent | b0e83d70f935664482cf4006f9bac8315d55c480 (diff) | |
| download | gcc-c96828f1ec5bb54524697bd0f3fb22088fda5342.zip gcc-c96828f1ec5bb54524697bd0f3fb22088fda5342.tar.gz gcc-c96828f1ec5bb54524697bd0f3fb22088fda5342.tar.bz2 | |
[rs6000] Fix PR92760 by checking VECTOR_MEM_NONE_P instead
PR92760 exposed one issue that VECTOR_UNIT_NONE_P (V2DImode) is true on Power7
then we won't return it as preferred_simd_mode but ISA 2.06 (Power7) does
introduce partial support on vector doubleword (very limitted) and more basic
support origins from ISA 2.07 (Power8) though. To make vectorizer still
leverage those few but available V2DImode related instructions, we need to
claim it's available on VSX (Power7 and up).
gcc/ChangeLog
PR target/92760
* gcc/config/rs6000/rs6000.c (rs6000_preferred_simd_mode): Use
VECTOR_MEM_NONE_P instead of VECTOR_UNIT_NONE_P.
From-SVN: r278955
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/config/rs6000/rs6000.c | 2 |
2 files changed, 7 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index cffe0fb..e04a48a 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2019-12-04 Kewen Lin <linkw@gcc.gnu.org> + + PR target/92760 + * gcc/config/rs6000/rs6000.c (rs6000_preferred_simd_mode): Use + VECTOR_MEM_NONE_P instead of VECTOR_UNIT_NONE_P. + 2019-12-03 Jan Hubicka <hubicka@ucw.cz> * ipa-fnsummary.c: Include tree-into-ssa.h. diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c index 3c22f64..23b6d99 100644 --- a/gcc/config/rs6000/rs6000.c +++ b/gcc/config/rs6000/rs6000.c @@ -4917,7 +4917,7 @@ rs6000_preferred_simd_mode (scalar_mode mode) { opt_machine_mode vmode = mode_for_vector (mode, 16 / GET_MODE_SIZE (mode)); - if (vmode.exists () && !VECTOR_UNIT_NONE_P (vmode.require ())) + if (vmode.exists () && !VECTOR_MEM_NONE_P (vmode.require ())) return vmode.require (); return word_mode; |