Re: [llvm-commits] CVS: llvm/lib/AsmParser/Lexer.l llvmAsmParser.y

2006-12-31 Thread Reid Spencer
Oops. Sorry, missed that. Its fixed and regenerated now. Reid. On Sun, 2006-12-31 at 13:43 -0800, Jeff Cohen wrote: > Only half-fixed. The extra semi-colon is still there. > > Reid Spencer wrote: > > Jeff, > > > > This is now fixed. I wish the newer versions of bison would at least > > warn ab

Re: [llvm-commits] CVS: llvm/lib/AsmParser/Lexer.l llvmAsmParser.y

2006-12-31 Thread Jeff Cohen
Only half-fixed. The extra semi-colon is still there. Reid Spencer wrote: > Jeff, > > This is now fixed. I wish the newer versions of bison would at least > warn about this! > > Reid. > > On Sun, 2006-12-31 at 12:42 -0800, Jeff Cohen wrote: > >> A semi-colon appears to be in the wrong place, a

Re: [llvm-commits] CVS: llvm/lib/AsmParser/Lexer.l llvmAsmParser.y

2006-12-31 Thread Reid Spencer
Jeff, This is now fixed. I wish the newer versions of bison would at least warn about this! Reid. On Sun, 2006-12-31 at 12:42 -0800, Jeff Cohen wrote: > A semi-colon appears to be in the wrong place, as marked below. Some > versions of bison complain, while others silently accept it (including

Re: [llvm-commits] CVS: llvm/lib/AsmParser/Lexer.l llvmAsmParser.y

2006-12-31 Thread Jeff Cohen
A semi-colon appears to be in the wrong place, as marked below. Some versions of bison complain, while others silently accept it (including the newest ones)--though how they interpret the grammar is not clear. Reid Spencer wrote: > Changes in directory llvm/lib/AsmParser: > > Lexer.l updated: 1