* combine.c (make_extraction): Correct offset computation.

	* gcc.c-torture/execute/20051012-1.c: New test.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@105330 138bc75d-0d04-0410-961f-82ee72b054a4

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 93da687..f2b9db3 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,7 @@
+2005-10-12  Adrian Straetling <straetling@de.ibm.com>
+
+       * combine.c (make_extraction): Correct offset computation.
+
 2005-10-12  Hans-Peter Nilsson  <hp@axis.com>
 
        * config/cris/t-linux (LIMITS_H_TEST): Define.

diff --git a/gcc/combine.c b/gcc/combine.c
index 3be34e4..ff10663 100644 (file)
--- a/gcc/combine.c
+++ b/gcc/combine.c
@@ -6512,11 +6512,12 @@ make_extraction (enum machine_mode mode, rtx inner, HOST_WIDE_INT pos,
        offset -= GET_MODE_SIZE (is_mode) - GET_MODE_SIZE (inner_mode);
 
       /* If this is a constant position, we can move to the desired byte.
-        Be careful not to go beyond the original object. */
+        Be careful not to go beyond the original object and maintain the
+        natural alignment of the memory.  */ 
       if (pos_rtx == 0)
        {
          enum machine_mode bfmode = smallest_mode_for_size (len, MODE_INT);
-         offset += pos / GET_MODE_BITSIZE (bfmode);
+         offset += (pos / GET_MODE_BITSIZE (bfmode)) * GET_MODE_SIZE (bfmode);
          pos %= GET_MODE_BITSIZE (bfmode);
        }
 

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 7abf169..be34d21 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2005-10-12  Adrian Straetling <straetling@de.ibm.com>
+
+       * gcc.c-torture/execute/20051012-1.c: New test.
+
 2005-10-12  Paul Thomas  <pault@gcc.gnu.org>
 
        PR fortran/18082

diff --git a/gcc/testsuite/gcc.c-torture/execute/20051012-1.c b/gcc/testsuite/gcc.c-torture/execute/20051012-1.c
new file mode 100644 (file)
index 0000000..f12f429
--- /dev/null
+++ b/gcc/testsuite/gcc.c-torture/execute/20051012-1.c
@@ -0,0 +1,24 @@
+extern void abort (void);
+
+struct type 
+{
+  int *a;
+  
+  int b:16;
+  unsigned int p:9;
+} t;
+
+unsigned int 
+foo ()
+{
+  return t.p;
+}
+
+int 
+main (void)
+{
+  t.p = 8;
+  if (foo (t) != 8)
+    abort ();
+  return 0;
+}