Closed Bug 927428 Opened 7 years ago Closed 7 years ago

use __LP64__ instead of HAVE_64BIT_OS in the linker

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla27

People

(Reporter: froydnj, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

No description provided.
I think we could dispense with the __SIZEOF_POINTER__ check, too?
Attachment #817863 - Flags: review?(mh+mozilla)
Blocks: 925167
Comment on attachment 817863 [details] [diff] [review]
use __LP64__ instead of HAVE_64BIT_OS in the linker

Review of attachment 817863 [details] [diff] [review]:
-----------------------------------------------------------------

::: mozglue/linker/Logging.h
@@ +67,5 @@
>        LOG(__VA_ARGS__);  \
>      }                    \
>    } while(0)
>  
> +#if defined(__LP64__) || __SIZEOF_POINTER__ == 8

You can remove the __SIZEOF_POINTER__ check, yes.
Attachment #817863 - Flags: review?(mh+mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/09fcb0805831
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.