DescriptionImprove x32 detection macro.
When targeting the Microsoft ABI in 64bit mode, clang defines __x86_64__ but
doesn't define __LP64__ (Microsoft uses LLP64), so it would fall down the x32
path. cl.exe doesn't define __x86_64__ in the first place, so it didn't have
this problem.
Rather than trying to guess pointer size by looking at __x86_64__ and __LP64__,
check for pointer size directly using __POINTER_SIZE__. This is defined by both
gcc and clang, and eliminiates this problem.
This should fix hundreds of "error(clang): unknown type name 'Atomic64'" when
compiling v8 on Windows with clang for 64 bit.
BUG=chromium:82385
LOG=n
R=haitao.feng@intel.com, jochen@chromium.org
Committed: https://code.google.com/p/v8/source/detail?r=23855
Patch Set 1 #
Total comments: 2
Patch Set 2 : #Messages
Total messages: 10 (3 generated)
|