re PR middle-end/53790 (ICE on dereferencing a extern union in asm statement)

2012-06-28  Richard Guenther  <rguenther@suse.de>

	PR middle-end/53790
	* expr.c (expand_expr_real_1): Verify if the type is complete
	before inspecting its size.

	* gcc.dg/torture/pr53790.c: New testcase.

From-SVN: r189045

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 8b9f9cb7c5220f168ec32cf9f22f2dd8c4818ca5..76e8913d6ae35cf790d340b5c6c375ba53d05ad8 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2012-06-28  Richard Guenther  <rguenther@suse.de>
+
+       PR middle-end/53790
+       * expr.c (expand_expr_real_1): Verify if the type is complete
+       before inspecting its size.
+
 2012-06-28  Andreas Schwab  <schwab@linux-m68k.org>
 
        * doc/include/gpl.texi: Remove.

diff --git a/gcc/expr.c b/gcc/expr.c
index 0d26442b500b4c8f1a423e3cde023ecf83bc6d5a..9a9af5b9903d906f6a7dd30ee72e3a8f9f1aa2df 100644 (file)
--- a/gcc/expr.c
+++ b/gcc/expr.c
@@ -9832,6 +9832,7 @@ expand_expr_real_1 (tree exp, rtx target, enum machine_mode tmode,
        orig_op0 = op0
          = expand_expr (tem,
                         (TREE_CODE (TREE_TYPE (tem)) == UNION_TYPE
+                         && COMPLETE_TYPE_P (TREE_TYPE (tem))
                          && (TREE_CODE (TYPE_SIZE (TREE_TYPE (tem)))
                              != INTEGER_CST)
                          && modifier != EXPAND_STACK_PARM

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index f7ca83089cc7cba3fbed23fae464d429b0b71bd6..406bd1f5dcf768c6bda515aef609f6b7dc195e6d 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2012-06-28  Richard Guenther  <rguenther@suse.de>
+
+       PR middle-end/53790
+       * gcc.dg/torture/pr53790.c: New testcase.
+
 2012-06-28  Jakub Jelinek  <jakub@redhat.com>
 
        PR tree-optimization/53645

diff --git a/gcc/testsuite/gcc.dg/torture/pr53790.c b/gcc/testsuite/gcc.dg/torture/pr53790.c
new file mode 100644 (file)
index 0000000..0abd3dd
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/torture/pr53790.c
@@ -0,0 +1,17 @@
+/* { dg-do compile } */
+
+typedef struct s {
+    int value;
+} s_t;
+
+static inline int 
+read(s_t const *var)
+{
+  return var->value;
+}
+
+int main()
+{
+  extern union u extern_var;
+  return read((s_t *)&extern_var);
+}