Commit 74aad7cc by Jakub Jelinek Committed by Jakub Jelinek

re PR c++/19263 (wrong-code: explicit initialization of v8qi miscompiled)

	PR c++/19263
	* typeck2.c (split_nonconstant_init_1) <case VECTOR_TYPE>: Put a copy
	of CONSTRUCTOR's node into MODIFY_EXPR, as the original is modified.

	* g++.dg/init/vector1.C: New test.

From-SVN: r93672
parent 28bbb550
2005-01-15 Jakub Jelinek <jakub@redhat.com>
PR c++/19263
* typeck2.c (split_nonconstant_init_1) <case VECTOR_TYPE>: Put a copy
of CONSTRUCTOR's node into MODIFY_EXPR, as the original is modified.
2005-01-12 Kaveh R. Ghazi <ghazi@caip.rutgers.edu> 2005-01-12 Kaveh R. Ghazi <ghazi@caip.rutgers.edu>
* Make-lang.in (cp-warn): Don't append $(WERROR). * Make-lang.in (cp-warn): Don't append $(WERROR).
......
...@@ -506,8 +506,9 @@ split_nonconstant_init_1 (tree dest, tree init) ...@@ -506,8 +506,9 @@ split_nonconstant_init_1 (tree dest, tree init)
case VECTOR_TYPE: case VECTOR_TYPE:
if (!initializer_constant_valid_p (init, type)) if (!initializer_constant_valid_p (init, type))
{ {
tree cons = copy_node (init);
CONSTRUCTOR_ELTS (init) = NULL; CONSTRUCTOR_ELTS (init) = NULL;
code = build2 (MODIFY_EXPR, type, dest, init); code = build2 (MODIFY_EXPR, type, dest, cons);
code = build_stmt (EXPR_STMT, code); code = build_stmt (EXPR_STMT, code);
add_stmt (code); add_stmt (code);
} }
......
2005-01-15 Jakub Jelinek <jakub@redhat.com>
PR c++/19263
* g++.dg/init/vector1.C: New test.
2005-01-14 James E. Wilson <wilson@specifixinc.com> 2005-01-14 James E. Wilson <wilson@specifixinc.com>
PR target/13158 PR target/13158
......
// PR c++/19263
// { dg-do run }
// { dg-options "-O2" }
typedef signed char v8qi __attribute__ ((vector_size (8)));
extern "C" void abort (void);
static unsigned char S[16];
struct A
{
int i;
v8qi j, k;
int l;
};
void
foo (unsigned char v)
{
A a = { 1, { v, v, v, v, v, v, v, v },
{ v + 1, v + 1, v + 1, v + 1, v + 1, v + 1, v + 1, v + 1 }, 3 };
v8qi *s = (v8qi *) &S[0];
*s = a.j;
s[1] = a.k;
}
void
bar (unsigned char v)
{
v8qi val8 = { v, v, v, v, v, v, v, v };
v8qi *s = (v8qi *) &S[0];
*s = val8;
}
int n = 5, cnt;
int
num (void)
{
++cnt;
return n;
}
void
baz (void)
{
static A a = { 0, { num (), num (), num (), num (), 6, 6, 6, 6 },
{ 7, 7, 7, 7, 8, 8, 8, 8 }, 0 };
v8qi *s = (v8qi *) &S[0];
*s = a.j;
s[1] = a.k;
}
int
main ()
{
int i;
foo (1);
for (i = 0; i < 8; ++i)
if (S[i] != 1)
abort ();
for (; i < 16; ++i)
if (S[i] != 2)
abort ();
bar (3);
for (i = 0; i < 8; ++i)
if (S[i] != 3)
abort ();
return 0;
baz ();
if (cnt != 4)
abort ();
for (i = 0; i < 16; ++i)
if (S[i] != 5 + (i / 4))
abort ();
return 0;
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment