[llvm-commits] [llvm] r44213 - in /llvm/trunk/lib/AsmParser: LLLexer.cpp LLLexer.h Lexer.cpp.cvs Lexer.l Lexer.l.cvs Makefile Parser.cpp ParserInternals.h llvmAsmParser.y

Chris Lattner clattner at apple.com
Sun Nov 18 13:24:20 PST 2007


On Nov 18, 2007, at 3:06 AM, Anton Korobeynikov wrote:

> Hello, Chris
>
>> Replace the original flex lexer with a hand writen one.  This
>> drops a dependency on flex and lets us make future progress more
>> easily.  Yay for 2 fewer .cvs files to make silly conflicts with.
> I don't know, was is due to your patch, but I have just noticed:
>
> llvm[2]: Bisoning llvmAsmParser.y
> /home/asl/proj/llvm/src/lib/AsmParser/llvmAsmParser.y: conflicts: 106
> shift/reduce
>
> This looks pretty insane (if I remember correctly, it was ~10
> conflicts).

This predated my patch.  This happened when Reid did some hacks for  
sext/zext attributes on calls, to disambiguate them from sext/zext  
instructions.

With luck, I may be able to replace the bison parsers, which will  
eliminate this issue.

-Chris



More information about the llvm-commits mailing list