927427 - use __LP64__ in FileUtils.cpp for Linux

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Nathan Froyd [:froydnj]]

I missed that Linux needed this conversion too, just like OSX.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: use __LP64__ in FileUtils.cpp for Linux

Comment 2

[Benjamin Smedberg]

Comment on attachment 817862 [details] [diff] [review]
use __LP64__ in FileUtils.cpp for Linux

Why does it need this conversion? I thought that the LP64 stuff was some mac-specific issue.

Comment 3

[Nathan Froyd [:froydnj]]

(In reply to Benjamin Smedberg  [:bsmedberg] from comment #2)
> Comment on attachment 817862 [details] [diff] [review]
> use __LP64__ in FileUtils.cpp for Linux
> 
> Why does it need this conversion? I thought that the LP64 stuff was some
> mac-specific issue.

__LP64__ is the "standard" unix-ish define for indicating the machine has 64-bit longs and 64-bit pointers.  So it works for indicating 64-bitness on GCC/Linux just as well as clang/OSX.

Comment 4

[Benjamin Smedberg]

Comment on attachment 817862 [details] [diff] [review]
use __LP64__ in FileUtils.cpp for Linux

ok... although aren't there 64-bit architectures which aren't lp64? ilp64, for example. It probably doesn't matter in practice, but I do wonder why we're bothering to make this change at all.

Comment 5

[Nathan Froyd [:froydnj]]

(In reply to Benjamin Smedberg  [:bsmedberg] from comment #4)
> Comment on attachment 817862 [details] [diff] [review]
> use __LP64__ in FileUtils.cpp for Linux
> 
> ok... although aren't there 64-bit architectures which aren't lp64? ilp64,
> for example. It probably doesn't matter in practice, but I do wonder why
> we're bothering to make this change at all.

The idea is to either (a) get rid of HAVE_64BIT_OS so that universal mac builds become easier; or (b) reduce HAVE_64BIT_OS to something only makefiles and win32 cares about.  But it's an open question how much we would care about faster universal mac builds (e.g. if we dropped 32-bit support).  I need to post to dev-planning about that.

Comment 6

[Nathan Froyd [:froydnj]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d3e014a0b5ad

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d3e014a0b5ad