diff options
Diffstat (limited to 'meta')
-rw-r--r-- | meta/recipes-devtools/gcc/gcc-7.3.inc | 1 | ||||
-rw-r--r-- | meta/recipes-devtools/gcc/gcc-7.3/0001-Fix-internal-compiler-error-in-testcase.patch | 212 |
2 files changed, 213 insertions, 0 deletions
diff --git a/meta/recipes-devtools/gcc/gcc-7.3.inc b/meta/recipes-devtools/gcc/gcc-7.3.inc index 7c835cf6b2..2dc3db8579 100644 --- a/meta/recipes-devtools/gcc/gcc-7.3.inc +++ b/meta/recipes-devtools/gcc/gcc-7.3.inc | |||
@@ -78,6 +78,7 @@ SRC_URI = "\ | |||
78 | ${BACKPORTS} \ | 78 | ${BACKPORTS} \ |
79 | " | 79 | " |
80 | BACKPORTS = "\ | 80 | BACKPORTS = "\ |
81 | file://0001-Fix-internal-compiler-error-in-testcase.patch \ | ||
81 | " | 82 | " |
82 | 83 | ||
83 | SRC_URI[md5sum] = "be2da21680f27624f3a87055c4ba5af2" | 84 | SRC_URI[md5sum] = "be2da21680f27624f3a87055c4ba5af2" |
diff --git a/meta/recipes-devtools/gcc/gcc-7.3/0001-Fix-internal-compiler-error-in-testcase.patch b/meta/recipes-devtools/gcc/gcc-7.3/0001-Fix-internal-compiler-error-in-testcase.patch new file mode 100644 index 0000000000..45cc0eeecd --- /dev/null +++ b/meta/recipes-devtools/gcc/gcc-7.3/0001-Fix-internal-compiler-error-in-testcase.patch | |||
@@ -0,0 +1,212 @@ | |||
1 | From d3cdd96a300f9003a1cc242541605169aacdc811 Mon Sep 17 00:00:00 2001 | ||
2 | From: willschm <willschm@138bc75d-0d04-0410-961f-82ee72b054a4> | ||
3 | Date: Mon, 25 Sep 2017 14:35:02 +0000 | ||
4 | Subject: [PATCH] [gcc] | ||
5 | |||
6 | 2017-09-25 Will Schmidt <will_schmidt@vnet.ibm.com> | ||
7 | |||
8 | * config/rs6000/rs6000.c (rs6000_gimple_fold_builtin): Add handling | ||
9 | for early folding of vector stores (ALTIVEC_BUILTIN_ST_*). | ||
10 | (rs6000_builtin_valid_without_lhs): New helper function. | ||
11 | * config/rs6000/rs6000-c.c (altivec_resolve_overloaded_builtin): | ||
12 | Remove obsoleted code for handling ALTIVEC_BUILTIN_VEC_ST. | ||
13 | |||
14 | git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@253152 138bc75d-0d04-0410-961f-82ee72b054a4 | ||
15 | |||
16 | Fix internal compiler error for testcase gcc.dg/vmx/7d-02.c | ||
17 | |||
18 | Upstream commit: d3cdd96a300f9003a1cc242541605169aacdc811 | ||
19 | |||
20 | Backport by Kaushik Phatak <Kaushik.Phatak@kpit.com> | ||
21 | |||
22 | This patch removes changes to Changelog from the original upstream patch. | ||
23 | This will help us avoid conflicts. | ||
24 | |||
25 | Upstream-Status: Backport | ||
26 | |||
27 | Signed-off-by: Mark Hatle <mark.hatle@windriver.com> | ||
28 | --- | ||
29 | gcc/ChangeLog | 8 +++++ | ||
30 | gcc/config/rs6000/rs6000-c.c | 72 ------------------------------------------- | ||
31 | gcc/config/rs6000/rs6000.c | 73 ++++++++++++++++++++++++++++++++++++++++++-- | ||
32 | 3 files changed, 78 insertions(+), 75 deletions(-) | ||
33 | |||
34 | diff --git a/gcc/config/rs6000/rs6000-c.c b/gcc/config/rs6000/rs6000-c.c | ||
35 | index a49db97..4a363a1 100644 | ||
36 | --- a/gcc/config/rs6000/rs6000-c.c | ||
37 | +++ b/gcc/config/rs6000/rs6000-c.c | ||
38 | @@ -6279,70 +6279,6 @@ altivec_resolve_overloaded_builtin (loca | ||
39 | } | ||
40 | } | ||
41 | |||
42 | - /* Similarly for stvx. */ | ||
43 | - if (fcode == ALTIVEC_BUILTIN_VEC_ST | ||
44 | - && (BYTES_BIG_ENDIAN || !VECTOR_ELT_ORDER_BIG) | ||
45 | - && nargs == 3) | ||
46 | - { | ||
47 | - tree arg0 = (*arglist)[0]; | ||
48 | - tree arg1 = (*arglist)[1]; | ||
49 | - tree arg2 = (*arglist)[2]; | ||
50 | - | ||
51 | - /* Construct the masked address. Let existing error handling take | ||
52 | - over if we don't have a constant offset. */ | ||
53 | - arg1 = fold (arg1); | ||
54 | - | ||
55 | - if (TREE_CODE (arg1) == INTEGER_CST) | ||
56 | - { | ||
57 | - if (!ptrofftype_p (TREE_TYPE (arg1))) | ||
58 | - arg1 = build1 (NOP_EXPR, sizetype, arg1); | ||
59 | - | ||
60 | - tree arg2_type = TREE_TYPE (arg2); | ||
61 | - if (TREE_CODE (arg2_type) == ARRAY_TYPE && c_dialect_cxx ()) | ||
62 | - { | ||
63 | - /* Force array-to-pointer decay for C++. */ | ||
64 | - arg2 = default_conversion (arg2); | ||
65 | - arg2_type = TREE_TYPE (arg2); | ||
66 | - } | ||
67 | - | ||
68 | - /* Find the built-in to make sure a compatible one exists; if not | ||
69 | - we fall back to default handling to get the error message. */ | ||
70 | - for (desc = altivec_overloaded_builtins; | ||
71 | - desc->code && desc->code != fcode; desc++) | ||
72 | - continue; | ||
73 | - | ||
74 | - for (; desc->code == fcode; desc++) | ||
75 | - if (rs6000_builtin_type_compatible (TREE_TYPE (arg0), desc->op1) | ||
76 | - && rs6000_builtin_type_compatible (TREE_TYPE (arg1), desc->op2) | ||
77 | - && rs6000_builtin_type_compatible (TREE_TYPE (arg2), | ||
78 | - desc->op3)) | ||
79 | - { | ||
80 | - tree addr = fold_build2_loc (loc, POINTER_PLUS_EXPR, arg2_type, | ||
81 | - arg2, arg1); | ||
82 | - tree aligned | ||
83 | - = fold_build2_loc (loc, BIT_AND_EXPR, arg2_type, | ||
84 | - addr, build_int_cst (arg2_type, -16)); | ||
85 | - | ||
86 | - tree arg0_type = TREE_TYPE (arg0); | ||
87 | - if (TYPE_MODE (arg0_type) == V2DImode) | ||
88 | - /* Type-based aliasing analysis thinks vector long | ||
89 | - and vector long long are different and will put them | ||
90 | - in distinct alias classes. Force our address type | ||
91 | - to be a may-alias type to avoid this. */ | ||
92 | - arg0_type | ||
93 | - = build_pointer_type_for_mode (arg0_type, Pmode, | ||
94 | - true/*can_alias_all*/); | ||
95 | - else | ||
96 | - arg0_type = build_pointer_type (arg0_type); | ||
97 | - aligned = build1 (NOP_EXPR, arg0_type, aligned); | ||
98 | - tree stg = build_indirect_ref (loc, aligned, RO_NULL); | ||
99 | - tree retval = build2 (MODIFY_EXPR, TREE_TYPE (stg), stg, | ||
100 | - convert (TREE_TYPE (stg), arg0)); | ||
101 | - return retval; | ||
102 | - } | ||
103 | - } | ||
104 | - } | ||
105 | - | ||
106 | for (n = 0; | ||
107 | !VOID_TYPE_P (TREE_VALUE (fnargs)) && n < nargs; | ||
108 | fnargs = TREE_CHAIN (fnargs), n++) | ||
109 | diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c | ||
110 | index a49db97..4a363a1 100644 | ||
111 | --- a/gcc/config/rs6000/rs6000.c | ||
112 | +++ b/gcc/config/rs6000/rs6000.c | ||
113 | @@ -55,6 +55,7 @@ | ||
114 | #include "reload.h" | ||
115 | #include "sched-int.h" | ||
116 | #include "gimplify.h" | ||
117 | +#include "gimple-fold.h" | ||
118 | #include "gimple-iterator.h" | ||
119 | #include "gimple-ssa.h" | ||
120 | #include "gimple-walk.h" | ||
121 | @@ -17089,6 +17090,25 @@ rs6000_fold_builtin (tree fndecl, int n_ | ||
122 | #endif | ||
123 | } | ||
124 | |||
125 | +/* Helper function to sort out which built-ins may be valid without having | ||
126 | + a LHS. */ | ||
127 | +bool | ||
128 | +rs6000_builtin_valid_without_lhs (enum rs6000_builtins fn_code) | ||
129 | +{ | ||
130 | + switch (fn_code) | ||
131 | + { | ||
132 | + case ALTIVEC_BUILTIN_STVX_V16QI: | ||
133 | + case ALTIVEC_BUILTIN_STVX_V8HI: | ||
134 | + case ALTIVEC_BUILTIN_STVX_V4SI: | ||
135 | + case ALTIVEC_BUILTIN_STVX_V4SF: | ||
136 | + case ALTIVEC_BUILTIN_STVX_V2DI: | ||
137 | + case ALTIVEC_BUILTIN_STVX_V2DF: | ||
138 | + return true; | ||
139 | + default: | ||
140 | + return false; | ||
141 | + } | ||
142 | +} | ||
143 | + | ||
144 | /* Fold a machine-dependent built-in in GIMPLE. (For folding into | ||
145 | a constant, use rs6000_fold_builtin.) */ | ||
146 | |||
147 | @@ -17102,6 +17122,10 @@ rs6000_gimple_fold_builtin (gimple_stmt_ | ||
148 | = (enum rs6000_builtins) DECL_FUNCTION_CODE (fndecl); | ||
149 | tree arg0, arg1, lhs; | ||
150 | |||
151 | + /* Prevent gimple folding for code that does not have a LHS, unless it is | ||
152 | + allowed per the rs6000_builtin_valid_without_lhs helper function. */ | ||
153 | + if (!gimple_call_lhs (stmt) && !rs6000_builtin_valid_without_lhs (fn_code)) | ||
154 | + return false; | ||
155 | switch (fn_code) | ||
156 | { | ||
157 | /* Flavors of vec_add. We deliberately don't expand | ||
158 | @@ -17169,6 +17193,54 @@ rs6000_gimple_fold_builtin (gimple_stmt_ | ||
159 | gsi_replace (gsi, g, true); | ||
160 | return true; | ||
161 | } | ||
162 | + /* Vector stores. */ | ||
163 | + case ALTIVEC_BUILTIN_STVX_V16QI: | ||
164 | + case ALTIVEC_BUILTIN_STVX_V8HI: | ||
165 | + case ALTIVEC_BUILTIN_STVX_V4SI: | ||
166 | + case ALTIVEC_BUILTIN_STVX_V4SF: | ||
167 | + case ALTIVEC_BUILTIN_STVX_V2DI: | ||
168 | + case ALTIVEC_BUILTIN_STVX_V2DF: | ||
169 | + { | ||
170 | + /* Do not fold for -maltivec=be on LE targets. */ | ||
171 | + if (VECTOR_ELT_ORDER_BIG && !BYTES_BIG_ENDIAN) | ||
172 | + return false; | ||
173 | + arg0 = gimple_call_arg (stmt, 0); /* Value to be stored. */ | ||
174 | + arg1 = gimple_call_arg (stmt, 1); /* Offset. */ | ||
175 | + tree arg2 = gimple_call_arg (stmt, 2); /* Store-to address. */ | ||
176 | + location_t loc = gimple_location (stmt); | ||
177 | + tree arg0_type = TREE_TYPE (arg0); | ||
178 | + /* Use ptr_type_node (no TBAA) for the arg2_type. | ||
179 | + FIXME: (Richard) "A proper fix would be to transition this type as | ||
180 | + seen from the frontend to GIMPLE, for example in a similar way we | ||
181 | + do for MEM_REFs by piggy-backing that on an extra argument, a | ||
182 | + constant zero pointer of the alias pointer type to use (which would | ||
183 | + also serve as a type indicator of the store itself). I'd use a | ||
184 | + target specific internal function for this (not sure if we can have | ||
185 | + those target specific, but I guess if it's folded away then that's | ||
186 | + fine) and get away with the overload set." | ||
187 | + */ | ||
188 | + tree arg2_type = ptr_type_node; | ||
189 | + /* POINTER_PLUS_EXPR wants the offset to be of type 'sizetype'. Create | ||
190 | + the tree using the value from arg0. The resulting type will match | ||
191 | + the type of arg2. */ | ||
192 | + gimple_seq stmts = NULL; | ||
193 | + tree temp_offset = gimple_convert (&stmts, loc, sizetype, arg1); | ||
194 | + tree temp_addr = gimple_build (&stmts, loc, POINTER_PLUS_EXPR, | ||
195 | + arg2_type, arg2, temp_offset); | ||
196 | + /* Mask off any lower bits from the address. */ | ||
197 | + tree aligned_addr = gimple_build (&stmts, loc, BIT_AND_EXPR, | ||
198 | + arg2_type, temp_addr, | ||
199 | + build_int_cst (arg2_type, -16)); | ||
200 | + gsi_insert_seq_before (gsi, stmts, GSI_SAME_STMT); | ||
201 | + /* The desired gimple result should be similar to: | ||
202 | + MEM[(__vector floatD.1407 *)_1] = vf1D.2697; */ | ||
203 | + gimple *g; | ||
204 | + g = gimple_build_assign (build2 (MEM_REF, arg0_type, aligned_addr, | ||
205 | + build_int_cst (arg2_type, 0)), arg0); | ||
206 | + gimple_set_location (g, loc); | ||
207 | + gsi_replace (gsi, g, true); | ||
208 | + return true; | ||
209 | + } | ||
210 | |||
211 | default: | ||
212 | break; | ||