compilecmp icon indicating copy to clipboard operation
compilecmp copied to clipboard

-obj did not do what the doc lead me to believe it should

Open dr2chase opened this issue 8 years ago • 7 comments
trafficstars

I typed: compilecmp -all -obj

I expected to see some object file sizes, but instead I saw this:

compilecmp   master   HEAD
14:00:39 copy tree at master (90b39f33c40e8d4ae085426313be69473f304e7d) to /usr/local/google/home/drchase/.compilecmp/90b39f33c40e8d4ae085426313be69473f304e7d
14:00:39 /usr/local/google/home/drchase/.compilecmp/90b39f33c40e8d4ae085426313be69473f304e7d/src/make.bash
14:01:26 copy tree at HEAD (f3c833e24bd9278603afa147a89b2b816e068ea5) to /usr/local/google/home/drchase/.compilecmp/f3c833e24bd9278603afa147a89b2b816e068ea5
14:01:26 /usr/local/google/home/drchase/.compilecmp/f3c833e24bd9278603afa147a89b2b816e068ea5/src/make.bash
before: /usr/local/google/home/drchase/.compilecmp/90b39f33c40e8d4ae085426313be69473f304e7d
after:  /usr/local/google/home/drchase/.compilecmp/f3c833e24bd9278603afa147a89b2b816e068ea5
benchstat -geomean  /tmp/824703058 /tmp/812777865
14:02:12 exit status 2

dr2chase avatar Jun 07 '17 18:06 dr2chase

AFK but I think the bug here is actually the terrible error message ("exit status 2"). On a hunch, what is your git version? Might be old enough that 'git worktree --detach' doesn't exist.

josharian avatar Jun 07 '17 18:06 josharian

git version 2.13.0.etc

Not quite latest, but close?

dr2chase avatar Jun 07 '17 18:06 dr2chase

Yes, that should be fine. I'll have to go through and improve all error messages. If you are blocked on this and want to dig, next step is finding out where the exit status 2 is coming from.

josharian avatar Jun 07 '17 19:06 josharian

I'm only semi-blocked; I'm taking a shot at verifying all the things for Heschi's DWARF CL.

dr2chase avatar Jun 07 '17 19:06 dr2chase

Hmmmm:

refresh compilebench
multiple sources, choose one of these commands and run it:
    go get -u github.com/josharian/compilebench
    drwxr-x--- 3 drchase eng 4096 Jun 7 12:06 /usr/local/google/home/drchase/work/gocode/src/github.com/josharian/compilebench
    go get -u golang.org/x/tools/cmd/compilebench
    drwxr-x--- 2 drchase eng 4096 May 12 13:19 /usr/local/google/home/drchase/work/gocode/src/golang.org/x/tools/cmd/compilebench

dr2chase avatar Jun 07 '17 20:06 dr2chase

But I also notice that compilebench has no "-obj" flag, seems like that is the problem.

dr2chase avatar Jun 07 '17 20:06 dr2chase

Aha. Yes, compilecmp needs an updated copy of compilebench to work. Not sure how best to test for that. You should delete github.com/josharian/compilebench and use golang.org/x/tools/cmd/compilebench. The former was only there so I could send pull requests to Russ, prior to migrating it to gerrit.

josharian avatar Jun 13 '17 18:06 josharian