This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][LTO] Fix PR48437


On Wed, 7 Dec 2011, Jakub Jelinek wrote:

> On Wed, Dec 07, 2011 at 03:52:35PM +0100, Richard Guenther wrote:
> > *** gcc/testsuite/gcc.dg/guality/pr48437.c	(revision 0)
> > --- gcc/testsuite/gcc.dg/guality/pr48437.c	(revision 0)
> > ***************
> > *** 0 ****
> > --- 1,15 ----
> > + /* PR lto/48437 */
> > + /* { dg-do run } */
> > + /* { dg-options "-g" } */
> > + 
> > + int i __attribute__((used));
> > + int main()
> > + {
> > +   volatile int i;
> > +   for (i = 3; i < 7; ++i)
> > +     {
> > +       extern int i; /* { dg-final { gdb-test 7 "i" "0" } } */
> > +       asm volatile ("" : : : "memory");
> 
> Do you really want to test it on line 7 ({ of after main())?

Oops - so much for adding the /* dg */ stuff late ...

> I'd expect you want to look at it on the line where asm volatile is,
> and make sure you can put a breakpoint on it:
> #include "../nop.h"
> early and use NOP instead of "" as the asm pattern.

Changed.

Thanks,
Richard.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]