Re: Is it a bug allowing to copy GIMPLE_ASM with labels?

2018-12-28 Thread Bin.Cheng
On Sat, Dec 29, 2018 at 3:42 PM Alexander Monakov wrote: > > On Sat, 29 Dec 2018, Bin.Cheng wrote: > > tracer-1.c: Assembler messages: > > tracer-1.c:16: Error: symbol `foo_label' is already defined > > > > Root cause is in tracer.c which duplicates basic block without > > checking if any GIMPLE_A

Re: Is it a bug allowing to copy GIMPLE_ASM with labels?

2018-12-28 Thread Alexander Monakov
On Sat, 29 Dec 2018, Bin.Cheng wrote: > tracer-1.c: Assembler messages: > tracer-1.c:16: Error: symbol `foo_label' is already defined > > Root cause is in tracer.c which duplicates basic block without > checking if any GIMPLE_ASM defines labels. > Is this a bug or invalid code? This is invalid co

Is it a bug allowing to copy GIMPLE_ASM with labels?

2018-12-28 Thread Bin.Cheng
Hi, Give below test: volatile int a, b, c; __attribute__((noinline)) int foo () { int i; for (i = 0; i < 1000; i++) { if (i % 17) a++; else b++; asm volatile( "foo_label:\n" "pushq %%rbx\n" "popq %%rbx\n" :"=m"(c) :"r"(i)

gcc-8-20181228 is now available

2018-12-28 Thread gccadmin
Snapshot gcc-8-20181228 is now available on ftp://gcc.gnu.org/pub/gcc/snapshots/8-20181228/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 8 SVN branch with the following options: svn://gcc.gnu.org/svn/gcc/branches/gcc-8

Re: For libgomp OpenACC entry points, redefine the "device" argument to "flags"

2018-12-28 Thread Thomas Schwinge
Hi! On Wed, 19 Dec 2018 22:56:05 +0100, I wrote: > On Wed, 19 Dec 2018 15:18:12 +0100, Jakub Jelinek wrote: > > On Wed, Dec 19, 2018 at 03:03:42PM +0100, Jakub Jelinek wrote: > > > On Wed, Dec 19, 2018 at 02:59:54PM +0100, Thomas Schwinge wrote: > > > > Right. For OpenACC, there's no "device" cl