Re: tsearch in core patch, for inclusion

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

In response to

Browse pgsql-hackers by date

  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?

Browse pgsql-patches by date

  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?