Switch default optimisation to -xO4 for Sun Workshop builds

RESOLVED FIXED in mozilla1.2beta

Status

P5
enhancement
RESOLVED FIXED
17 years ago
14 years ago

People

(Reporter: roland.mainz, Assigned: netscape)

Tracking

Trunk
mozilla1.2beta
Sun
Solaris

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
RFE: Switch default optimisation for Sun Workshop builds to -xO4.

I have tested some Sun Workshop 6U2 build the last two weeks... looks stable for
me...
Priority: -- → P5
Target Milestone: --- → mozilla1.0

Comment 1

17 years ago
Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 
(you can query for this string to delete spam or retrieve the list of bugs I've 
moved)
Target Milestone: mozilla1.0 → mozilla1.0.1
(Reporter)

Comment 2

16 years ago
cls:
Can we run the "nebiros"-tinderbox some days with -xO4 and look if it causes any
problems ?
Flipped.  We should state our minimum recommended version of Forte somewhere. 
Right now I'm leaning towards 6U2 if only because it's the only version that
we've regularly built with for some time now.
Target Milestone: mozilla1.0.1 → mozilla1.2alpha
(Reporter)

Comment 4

16 years ago
Christopher Seawood wrote:
> Flipped.

Where ? I can't see the change in configure.in ... ;-(
"Flipped" referred to the suggestion of putting it on nebiros.  I don't plan on
checking it into configure.in until we have a minimum stated version of Forte
that we support so that when things start breaking with older versions (ie,
WS5), we can mark those as unsupported.
(Reporter)

Comment 6

16 years ago
cls:
I am not sure but I guess WS5 was EOL'ed some time ago... can anyone confirm
this (petez?) ?

Current WS6U2 compiles with -xO4 and -xO5 without problems, IMHO we should make
this the minimum Forte release we support.
(Reporter)

Comment 7

16 years ago
nidheesh:
1. What's the mimimum Workshop version supported by Sun ?
2. What do you think about bumping the default optimisation to -xO4 ?

Comment 8

16 years ago
I agree that Forte 6U2 with -xO5 is best. Sun build it's Solaris netscape using
F6U1 due to common build environment restrictions for Solaris 9. With F6U1 -xO5
works fine with sparc but needed a patch to build on solaris/intel. As long as
you have these patches I feel safe that one can use -x05 with even F6U1.

I am not sure if this patch is needed for F6U2. We have had test builds that
worked fine with F6U2 with -xO5 on sparc. I am not sure if U2 requires any patch
for intel builds. Once it is verified I think U2 with -xO5 is right way to go. 

Added Kevin Lee from Sun RE. He knows the exact patches used by Sun releases.
(Reporter)

Comment 9

16 years ago
Created attachment 96509 [details] [diff] [review]
Patch for 2002-08-18-08-trunk to switch default optimisation from -O to -xO4
Status: NEW → ASSIGNED
Target Milestone: mozilla1.2alpha → mozilla1.2beta
Comment on attachment 96509 [details] [diff] [review]
Patch for 2002-08-18-08-trunk to switch default optimisation from -O to -xO4

r=cls
Attachment #96509 - Flags: review+
Patch has been checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.