Two small ecj tweaks

[Andrew John Hughes]

On 01/12/2008, Andrew John Hughes <gnu_andrew at member.fsf.org> wrote:
> On 01/12/2008, Andrew Haley <aph at redhat.com> wrote:
>  > Andrew John Hughes wrote:
>  >  > On 01/12/2008, Andrew John Hughes <gnu_andrew at member.fsf.org> wrote:
>  >  >> On 01/12/2008, Andrew Haley <aph at redhat.com> wrote:
>  >  >>  > Andrew John Hughes wrote:
>  >  >>  >  > On 30/11/2008, Mark Wielaard <mark at klomp.org> wrote:
>  >  >>  >  >> Hi,
>  >  >>  >  >>
>  >  >>  >  >>  Two tweaks to make sure we have a fast bootstrap ecj available. This
>  >  >>  >  >>  makes sure that when we compile the ecj jar with gcj we use
>  >  >>  >  >>  -findirect-dispatch so any unresolved references in the jar itself don't
>  >  >>  >  >>  make the compilation fail (like what would happen with the
>  >  >>  >  >>  eclipse-ecj.jar from fedora 10). Also if we don't have a native-ecj (not
>  >  >>  >  >>  configured --with-gcj), but we did detect an ecj binary then use that
>  >  >>  >  >>  first before falling back on full interpretation with gij.
>  >  >>  >  >>
>  >  >>  >  >>  2008-11-30  Mark Wielaard  <mark at klomp.org>
>  >  >>  >  >>
>  >  >>  >  >>     * Makefile.am (stamps/native-ecj.stamp): Use -findirect-dispatch.
>  >  >>  >  >>     * javac.in: Use ecj binary if available and no native-ecj.
>  >  >>  >  >>
>  >  >>  >  >>  Committed and pushed,
>  >  >>  >  >>
>  >  >>  >  >>
>  >  >>  >  >>  Mark
>  >  >>  >  >>
>  >  >>  >  >>
>  >  >>  >  >
>  >  >>  >  > Thanks for this.  I still wonder why we detect javac on ecj builds but
>  >  >>  >  > don't use it any more -- too many issues in the past I guess.
>  >  >>  >  >
>  >  >>  >  > I would also suggest possibly adding -Dgnu.gcj.precompiled.db.path to
>  >  >>  >  > the interpreted invocation.  On at least Debian and Gentoo, the
>  >  >>  >  > mapping between ecj.jar and ecj.so is stored in the default database
>  >  >>  >  > but as far as I can see, it doesn't get used unless this option is
>  >  >>  >  > given.
>  >  >>  >
>  >  >>  >
>  >  >>  > It certainly should.  If you have an example of any case where this fails, please
>  >  >>  >  let me know and I'll follow it up with the maintainers.
>  >  >>  >
>  >  >>  >  I'd like a login so I can see why this isn't working.
>  >  >>  >
>  >  >>  >
>  >  >>  >  Andrew.
>  >  >>  >
>  >  >>
>  >  >>
>  >  >> Well here's the straces for each:
>  >  >>
>  >  >>  strace -f -o out.nodb /usr/lib/jvm/gcj-jdk/bin/java -cp
>  >  >>  /usr/share/eclipse-ecj-3.3/lib/ecj.jar
>  >  >>  org.eclipse.jdt.internal.compiler.batch.Main -1.5 Test.java
>  >  >>
>  >  >>  strace -f -o out.db /usr/lib/jvm/gcj-jdk/bin/java
>  >  >>  -Dgnu.gcj.precompiled.db.path=/usr/lib64/gcj-4.3.3-9/classmap.db -cp
>  >  >>  /usr/share/eclipse-ecj-3.3/lib/ecj.jar
>  >  >>  org.eclipse.jdt.internal.compiler.batch.Main -1.5 Test.java
>  >  >>
>  >  >>  Only out.db includes:
>  >  >>
>  >  >>  out.db:16889 open("/usr/lib64/native_ecj-3.3.so", O_RDONLY) = 5
>  >  >>
>  >  >>  There is no reference to it in out.nodb.
>  >
>  >
>  > Please use strace -etrace=file -s 99999999
>  >
>  >
>  >  Andrew.
>  >
>
>
> Ah that would make things easier :)
>
>  With that and the diff, I've spotted the issue and it does seem like a
>  broken gcj being built by Gentoo:
>
>  - open("/usr/share/java/classmap.gcjdb", O_RDONLY) = -1 ENOENT (No
>  such file or directory)
>  - open("/usr/share/locale/locale.alias", O_RDONLY) = 4
>  - open("/usr/share/locale/en_GB.UTF-8/LC_MESSAGES/libc.mo", O_RDONLY)
>  = -1 ENOENT (No such file or directory)
>  - open("/usr/share/locale/en_GB.utf8/LC_MESSAGES/libc.mo", O_RDONLY) =
>  -1 ENOENT (No such file or directory)
>  - open("/usr/share/locale/en_GB/LC_MESSAGES/libc.mo", O_RDONLY) = 4
>  - open("/usr/share/locale/en.UTF-8/LC_MESSAGES/libc.mo", O_RDONLY) =
>  -1 ENOENT (No such file or directory)
>  - open("/usr/share/locale/en.utf8/LC_MESSAGES/libc.mo", O_RDONLY) = -1
>  ENOENT (No such file or directory)
>  - open("/usr/share/locale/en/LC_MESSAGES/libc.mo", O_RDONLY) = -1
>  ENOENT (No such file or directory)
>  - statfs("/selinux", 0x7fff451155f0) = -1 ENOENT (No such file or directory)
>  - open("/proc/mounts", O_RDONLY)    = 4
>  - open("/tmp/ffir42O08", O_RDWR|O_CREAT|O_EXCL, 0600) = 4
>  - unlink("/tmp/ffir42O08")          = 0
>  + open("/usr/lib64/gcj-4.3.3-9/classmap.db", O_RDONLY) = 4
>  + stat("/usr/lib64/gcj-4.3.3-9/classmap.db", {st_mode=S_IFREG|0644,
>  st_size=1441792, ...}) = 0
>
> + open("/usr/lib64/native_ecj-3.3.so", O_RDONLY) = 5
>
> + open("/etc/ld.so.cache", O_RDONLY) = 5
>  + open("/usr/lib/gcc/x86_64-pc-linux-gnu/4.3.3/libgcj_bc.so.1", O_RDONLY) = 5
>
>  gcj-dbtool is reporting /usr/lib64/gcj-4.3.3-9/classmap.db as the
>  default db file, but the default being used by gcj is
>  /usr/share/java/classmap.gcjdb
>
>
>  --
>
> Andrew :-)
>
>  Support Free Java!
>  Contribute to GNU Classpath and the OpenJDK
>  http://www.gnu.org/software/classpath
>  http://openjdk.java.net
>
>  PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
>  Fingerprint: F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8
>
>

http://bugs.gentoo.org/show_bug.cgi?id=249475
-- 
Andrew :-)

Support Free Java!
Contribute to GNU Classpath and the OpenJDK
http://www.gnu.org/software/classpath
http://openjdk.java.net

PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
Fingerprint: F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8