[PATCH] D11618: [CMake] Pass -march=i686 when targeting i686 (PR24222)

Hans Wennborg hans at chromium.org
Wed Jul 29 18:31:41 PDT 2015


hans added inline comments.

================
Comment at: cmake/config-ix.cmake:183
@@ -182,3 +182,3 @@
       # be gone after we resolve PR14109.
-      test_target_arch(i686 __i686__ "-m32")
+      test_target_arch(i686 __i686__ "-m32" "-march=i686")
       test_target_arch(i386 __i386__ "-m32")
----------------
samsonov wrote:
> Please mention PR24222 in a comment here.
Will do.

================
Comment at: test/sanitizer_common/TestCases/Linux/open_memstream.cc:2
@@ -3,1 +1,3 @@
+// RUN: %clangxx -m64 -march='' -O0 -g -xc++ %s -o %t && %run %t
+// RUN: %clangxx -m64 -march='' -O3 -g -xc++ %s -o %t && %run %t
 // REQUIRES: x86_64-supported-target
----------------
samsonov wrote:
> Would `-march=x86_64` work here?
Yes, but it has to be spelled -march=x86-64. Would you prefer that?


http://reviews.llvm.org/D11618







More information about the llvm-commits mailing list