[LLVMbugs] [Bug 20896] New: Don't define __x86_64__ in microsoft mode?

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=20896

            Bug ID: 20896
           Summary: Don't define __x86_64__ in microsoft mode?
           Product: clang
           Version: unspecified
          Hardware: PC
                OS: All
            Status: NEW
          Severity: normal
          Priority: P
         Component: Frontend
          Assignee: unassignedclangbugs at nondot.org
          Reporter: nicolasweber at gmx.de
                CC: llvmbugs at cs.uiuc.edu
    Classification: Unclassified

v8 uses this technique to check for x32:

#if defined(__x86_64__) && !defined(__LP64__)
...
#endif

With cl.exe, this works as cl.exe doesn't define __x86_64__. With clang-cl,
this fails since clang-cl does define __x86_64__ but doesn't define __LP64__
(because Windows is LLP64).

clang-cl probably shouldn't define __x86_64__? It already defines the
cl-equivalent _M_X64.

https://codereview.chromium.org/18014003/diff/161001/src/base/build_config.h

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20140910/611a135a/attachment.html>