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: [email protected]
          Reporter: [email protected]
                CC: [email protected]
    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.
_______________________________________________
LLVMbugs mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs

Reply via email to