Closed Bug 798243 Opened 7 years ago Closed 7 years ago

Codesighs produces "Z: 0B" on Linux64 non-PGO

Categories

(Testing :: General, defect)

x86_64
Linux
defect
Not set

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: philor, Unassigned)

References

Details

(Whiteboard: [codesighs])

On any tree where we run non-PGO Linux64 builds (and where we have codesighs enabled, which will not include try after bug 798225 and before bug 798229), take a look at the build log or the line in the bottom right part of tbpl when the B is selected - for the current codesize, we report "Z: 0B" which is surprisingly lean and mean. For Linux32 non-PGO, we report a number (~45MB), for Linux32 PGO we report a number (~53MB), for Linux64 PGO we report a number (~143MB), but whether or not any of those number are reasonable and have actual meaning, they look much more meaningful than Linux64 non-PGO, 0B.
The values codesighs return have been useless for a very long time. It should be replaced with something else.
confusion alert - checked something into inbound with this bug number when the correct bug number was 798423. Sorry!
(In reply to Mike Hommey [:glandium] from comment #1)
> The values codesighs return have been useless for a very long time. It
> should be replaced with something else.

Conveniently, Ted already wrote that something else, in bug 659950, it's just taken so long to deploy that I'd completely forgetten it, despite having seen the code while spending two hours filing multiple bugs about the brokenness of codesighs.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
(In reply to Patrick McManus [:mcmanus] from comment #2)
> confusion alert - checked something into inbound with this bug number when
> the correct bug number was 798423. Sorry!

Without a matching backout in the same merge (to fix the bug number), the script we use will just mark bugs regardless; philor luckily happened to spot this.
Assignee: mcmanus → nobody
Depends on: 819044
Depends on: 823030
(Removing dependant bugs intended for bug 798423)
No longer depends on: 819044, 823030
You need to log in before you can comment on or make changes to this bug.