On Sun, 6 Dec 2015, Bernd Edlinger wrote: > > Hi, > > while looking at the handling of basic asm statements > I noticed two minor glitches, which I want to fix now. > > First there is a missing check in compare_gimple_asm in ipa-icf-gimple.c > > Here we check if two asm statements are exactly identical, > there is a possibility that one is a basic asm and the other is an > extended asm with zero operands. Even if both have the same string > the string means something slightly different, if % or { } are around. > > example: > > asm("%"); // OK > asm("%":); // error: invalid 'asm': invalid %-code > > > Boot-strapped and reg-tested on x86_64-pc-linux-gnu, > OK for trunk?
Ok. Richard.