Closed Bug 550154 Opened 10 years ago Closed 10 years ago

PGO build issue on Solaris SPARC

Categories

(Firefox Build System :: General, defect)

Sun
OpenSolaris
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9.3a3

People

(Reporter: ginnchen+exoracle, Assigned: ginnchen+exoracle)

Details

Attachments

(2 files)

There're 2 issues on SPARC
1)
In Bug 425626, we disable pgo for xpcom/reflect/xptcall/src/md/unix on SPARC
The fix doesn't work now.
We need to move NO_PROFILE_GUIDED_OPTIMIZE ahead of include config.mk

2)
-xlinkopt=2 option failed to compile libmozjs.so
Might be a bug of compiler.
We'd remove this option.
Attachment #430265 - Flags: review?(ted.mielczarek)
Status: NEW → ASSIGNED
Attachment #430265 - Flags: review?(ted.mielczarek) → review+
http://hg.mozilla.org/mozilla-central/rev/d659d4463ab1
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Flags: in-testsuite-
Target Milestone: --- → mozilla1.9.3a3
Serge, I don't think you need to add test "$MOZILLA_1_9_2_BRANCH" = "1";.
This patch should be applied to both 1.9.2 and 1.9.3.
Attachment #435068 - Flags: review?(bugspam.Callek) → review+
(In reply to comment #4)
> Serge, I don't think you need to add test "$MOZILLA_1_9_2_BRANCH" = "1";.
> This patch should be applied to both 1.9.2 and 1.9.3.

Ginn, are you willing to test TB with a mozilla-1.9.2 tree with a version of this patch without the 1.9.2 test he did here?

We do the test to try and keep the build system in sync for the various mozilla-branches (where it makes sense). I'm just not sure if its worth asking him to write a new patch here if SPARC will fail even with this for TB on 1.9.2 (note: suite doesn't support 1.9.2)

Also no c-c app officially supports PGO at the moment anyway, though we are _trying_ to keep build-system support intact.
(In reply to comment #5)
> I'm just not sure if its worth asking him to write a new patch...

If its tested to work; and patch is written I'm not opposed to taking it though. Just to be clear.
Attachment #430265 - Attachment description: patch → patch [Checkin: Comment 2]
Comment on attachment 435068 [details] [diff] [review]
(Bv1-CC) Copy it to comm-central
[Checkin: Comment 7]


http://hg.mozilla.org/comm-central/rev/a4617f9d05e3


(In reply to comment #4)
> This patch should be applied to both 1.9.2 and 1.9.3.

In other words, you should land your fix in m-1.9.2 first.!.
Attachment #435068 - Attachment description: (Bv1-CC) Copy it to comm-central → (Bv1-CC) Copy it to comm-central [Checkin: Comment 7]
> (In reply to comment #4)
> > This patch should be applied to both 1.9.2 and 1.9.3.
> 
> In other words, you should land your fix in m-1.9.2 first.!.

I'd like to, but I guess drivers are not glad to take tier-n platform changes to stable branches.
e.g. Bug 548218, it blocks the build on Solaris, but didn't get approval yet.
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.