Bug 181133 - generate_offset_extractor.rb should not print to stderr by default
Summary: generate_offset_extractor.rb should not print to stderr by default
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-12-22 11:53 PST by Michael Catanzaro
Modified: 2017-12-22 12:31 PST (History)
9 users (show)

See Also:


Attachments
Patch (1.39 KB, patch)
2017-12-22 11:55 PST, Michael Catanzaro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2017-12-22 11:53:40 PST
Currently we have a build that is completely quiet between CMake time and install time, with the sole exception of generate_offset_extractor.rb, which always prints one line to stderr:

[206/2826] Generating ../../DerivedSou...s/JavaScriptCore/LLIntDesiredOffsets.h
Only dealing with backends: ["X86_64"]

It doesn't seem to be adding any value. Can we just get rid of it?
Comment 1 Michael Catanzaro 2017-12-22 11:55:00 PST
Created attachment 330135 [details]
Patch
Comment 2 Mark Lam 2017-12-22 12:06:49 PST
(In reply to Michael Catanzaro from comment #0)
> Currently we have a build that is completely quiet between CMake time and
> install time, with the sole exception of generate_offset_extractor.rb, which
> always prints one line to stderr:
> 
> [206/2826] Generating
> ../../DerivedSou...s/JavaScriptCore/LLIntDesiredOffsets.h
> Only dealing with backends: ["X86_64"]
> 
> It doesn't seem to be adding any value. Can we just get rid of it?

Fil added this back in r172777. I suspect it is to aid in debugging whether the offlineasm successfully generated the expected backends (this is meaningful for builds that generate code for supporting multiple backends).

That said, I think that code in the offlineasm has been stable for years now, and it isn't that hard to add this print back should we need to debug this in the future.  So, I do not object to removing it.
Comment 3 WebKit Commit Bot 2017-12-22 12:30:27 PST
Comment on attachment 330135 [details]
Patch

Clearing flags on attachment: 330135

Committed r226270: <https://trac.webkit.org/changeset/226270>
Comment 4 WebKit Commit Bot 2017-12-22 12:30:28 PST
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2017-12-22 12:31:26 PST
<rdar://problem/36197253>