[gcjx] Patch: FYI: parser and lexer changes

[Ranjit Mathew]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Tom Tromey wrote:
> 
> This weekend I looked into gcjx performance a bit.  The appended patch
> yields about a 30% speed improvement when building Classpath on my
> machine.  This is nice, but not nearly good enough; I think I'll

You call a 30% speedup not good enough? Whoa!


> probably just have to throw out the current parser and write one using
> a real tool.  Most likely, I'll put this off for a while.

People seem to be moving from tool-based to hand-written
recursive-descent parsers (witness the C and C++ front-ends
in mainline). Why do you think the current parser is beyond
redemption?


> FWIW in the end I didn't use oprofile but instead built a static gcjx
> with -pg.  That seemed to yield the most useful results.

Agreed.

BTW, I applied the attached patch as obvious to restore the
caret-printing feature. Now I see:
- -------------------------- 8< --------------------------
~/src/tmp > cat Hello.java
public class Hello
{
  public static void main( String[] args)
  {
    junk t;
    System.out.println( "Hello World!");
  }
}
~/src/tmp > $GCJX Hello.java
./Hello.java:5:4: error: type named 'junk' is undefined

    junk t;
    ^

- -------------------------- 8< --------------------------

Yeah!

Thanks,
Ranjit.

- --
Ranjit Mathew       Email: rmathew AT gmail DOT com

Bangalore, INDIA.     Web: http://ranjitmathew.hostingzero.com/




-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFDQe27Yb1hx2wRS48RAudnAJ4wwVWUVvmeYMr+oc2Y/jrCKiclXgCff34r
2y+FvH3SuqT7QgxuoJTqUWw=
=CMYv
-----END PGP SIGNATURE-----
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: t1.txt
URL: <http://gcc.gnu.org/pipermail/java-patches/attachments/20051004/8472ba8d/attachment.txt>