Re: [PATCH] reimplement -fstrict-volatile-bitfields v4, part 1/2

[Sandra Loosemore <sandra at codesourcery dot com>]

On 10/29/2013 02:51 AM, Bernd Edlinger wrote:
> On Mon, 28 Oct 2013 21:29:24, Sandra Loosemore wrote:
> > I again tried backporting the patch series along with your fix to GCC
> > 4.8 and tested on arm-none-eabi. I found that it was still getting
> > stuck in infinite recursion unless the test from this patch hunk
>
> hmmm....
> Actually I used your path on a clean 4.8.2 and built for --target=arm-eabi.
> I have seen the recursion on the extract_bit_field, but not on store_bit_field
> so far, maybe you could give me a hint which test case exposes the other
> flavour of this recursion problem.

Sorry, I was going to describe that in more detail but I forgot.  It was 
compiling pr56997-1.c with -mthumb, like:

arm-none-eabi-gcc /path/to/gcc/testsuite/gcc.dg/pr56997-1.c -c 
-fno-diagnostics-show-caret   -fstrict-volatile-bitfields 
-DSTACK_SIZE=16384 -mthumb

Other testing with -mthumb looked fine.

> > Either way, still needs more work, and more thorough testing (more
> > targets, and obviously trunk as well as the 4.8 backport) before I'd
> > consider this ready to commit. I might or might not be able to find
> > some more time to work on this in the next week....
>
> Yes.
>
> And it would be good to reach Richard's Nov-21 deadline for GCC-4.9

Yeah, I know the clock is ticking.  :-(

-Sandra