Re: [PATCH] PR52528, combine fix

2012-03-10 Thread Eric Botcazou
> So the fix is a one-liner that sets subst_low_luid before the > expand_field_assignment() call. Bootstrapped and tested under i686, > x86-64, powerpc64. Cross-tested on ARM. I was a bit weary that some > optimization regression might appear, which will complicate things, but > everything looks fi

[PATCH] PR52528, combine fix

2012-03-10 Thread Chung-Lin Tang
Hi, As described in the PR, a testcase compiled for PowerPC: struct S { unsigned a : 30; unsigned b : 2; }; int foo (int b) { struct S s = {0}; s.b = b; return bar (0x000b0010, 0x00040100ULL, *(unsigned long *)&s); } currently this is compiled to: foo: lis 6,0x4 li 5,0