[illumos-Developer] [REVIEW] 780, 783, 784 gcc build fixes

Dan McDonald danmcd at nexenta.com
Wed Mar 9 11:55:55 PST 2011


On Wed, Mar 09, 2011 at 02:48:03PM -0500, Richard Lowe wrote:
> 
> Dan, I didn't nest the cpr_is_foo checks in a non-ifdef'd condition, as
> you'd asked because I wasn't sure if my response regarding the effectively
> constant nature of cpr_sleeptype made you happy enough or not.

Yeah... it actually did.  The constant in question is referenced in other
bits of non-ifdef-SPARC code, but none of them call unknown functions like
the bits your fixing.

> These changes are sufficient to build with gcc as the primary compiler, and
> boot the resulting bits 32bit in virtualbox.  I don't believe there are
> hardwired uses of Studio cc in the x86 build, as there are on SPARC, but
> haven't checked yet.  I have not had the opportunity to test
> platform-specifics on anything but virtualbox as yet (that is, no 64bit, no
> non-p123 pcbe, etc.).  If you try using gcc as the primary, you may still
> find more bugs.

But these are an improvement over what was.  Count me as an approver!

Dan



More information about the Developer mailing list