From: | Brian Hurt <bhurt(at)janestcapital(dot)com> |
---|---|
To: | Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: tsearch in core patch, for inclusion |
Date: | 2007-02-21 14:49:29 |
Message-ID: | 45DC5BF9.4020102@janestcapital.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Markus Schiltknecht wrote:
> Hi,
>
> I recall having read something about rewriting the parser. Together
> with Tom being worried about parser performance and knowing GCC has
> switched to a hand written parser some time ago, I suspected bison to
> be slow. That's why I've asked.
>
This has little to do with performance and everything to do with the
insanity which is C++:
http://gnu.teleglobe.net/software/gcc/gcc-3.4/changes.html
> * A hand-written recursive-descent C++ parser has replaced the
> YACC-derived C++ parser from previous GCC releases. The new
> parser contains much improved infrastructure needed for better
> parsing of C++ source codes, handling of extensions, and clean
> separation (where possible) between proper semantics analysis
> and parsing. The new parser fixes many bugs that were found in
> the old parser.
>
Short form: C++ is basically not LALR(1) parseable.
Brian
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2007-02-21 14:54:13 | Re: tsearch in core patch, for inclusion |
Previous Message | Peter Eisentraut | 2007-02-21 14:38:41 | Re: --enable-xml instead of --with-libxml? |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2007-02-21 14:54:13 | Re: tsearch in core patch, for inclusion |
Previous Message | Peter Eisentraut | 2007-02-21 14:38:41 | Re: --enable-xml instead of --with-libxml? |