r304085 - baremetal.cpp: Fix tests where lib suffix is 64

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Sun May 28 01:15:42 2017
New Revision: 304085

URL: http://llvm.org/viewvc/llvm-project?rev=304085&view=rev
Log:
baremetal.cpp: Fix tests where lib suffix is 64


Modified:
cfe/trunk/test/Driver/baremetal.cpp

Modified: cfe/trunk/test/Driver/baremetal.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/baremetal.cpp?rev=304085&r1=304084&r2=304085&view=diff
==
--- cfe/trunk/test/Driver/baremetal.cpp (original)
+++ cfe/trunk/test/Driver/baremetal.cpp Sun May 28 01:15:42 2017
@@ -5,13 +5,13 @@
 // RUN: --sysroot=%S/Inputs/baremetal_arm \
 // RUN:   | FileCheck --check-prefix=CHECK-V6M-C %s
 // CHECK-V6M-C: "[[PREFIX_DIR:.*]]{{[/\\]+}}{{[^/^\\]+}}{{[/\\]+}}clang{{.*}}" 
"-cc1" "-triple" "thumbv6m-none--eabi"
-// CHECK-V6M-C-SAME: "-resource-dir" 
"[[PREFIX_DIR]]{{[/\\]+}}lib{{[/\\]+}}clang{{[/\\]+}}[[VERSION:[^"]*]]"
+// CHECK-V6M-C-SAME: "-resource-dir" 
"[[PREFIX_DIR]]{{[/\\]+}}lib{{(64)?}}{{[/\\]+}}clang{{[/\\]+}}[[VERSION:[^"]*]]"
 // CHECK-V6M-C-SAME: "-isysroot" "[[SYSROOT:[^"]*]]"
 // CHECK-V6M-C-SAME: "-internal-isystem" 
"[[SYSROOT]]{{[/\\]+}}include{{[/\\]+}}c++{{[/\\]+}}v1"
 // CHECk-V6M-C-SAME: "-internal-isystem" "[[SYSROOT]]{{[/\\]+}}include"
 // CHECK-V6M-C-SAME: "-x" "c++" "{{.*}}baremetal.cpp"
 // CHECK-V6M-C-NEXT: "{{[^"]*}}ld.lld" "{{.*}}.o" "-Bstatic"
-// CHECK-V6M-C-SAME: 
"-L[[PREFIX_DIR]]{{[/\\]+}}lib{{[/\\]+}}clang{{[/\\]+}}[[VERSION]]{{[/\\]+}}lib{{[/\\]+}}baremetal"
+// CHECK-V6M-C-SAME: 
"-L[[PREFIX_DIR]]{{[/\\]+}}lib{{(64)?}}{{[/\\]+}}clang{{[/\\]+}}[[VERSION]]{{[/\\]+}}lib{{[/\\]+}}baremetal"
 // CHECK-V6M-C-SAME: "-T" "semihosted.lds" 
"-Lsome{{[/\\]+}}directory{{[/\\]+}}user{{[/\\]+}}asked{{[/\\]+}}for"
 // CHECK-V6M-C-SAME: "-lc" "-lm" "-lclang_rt.builtins-armv6m.a"
 // CHECK-V6M-C-SAME: "-o" "{{.*}}.o"
@@ -33,7 +33,7 @@
 // RUN: --sysroot=%S/Inputs/baremetal_arm \
 // RUN:   | FileCheck --check-prefix=CHECK-V6M-DEFAULTCXX %s
 // CHECK-V6M-DEFAULTCXX: "{{[^"]*}}ld.lld" "{{.*}}.o" "-Bstatic"
-// CHECK-V6M-DEFAULTCXX-SAME: 
"-L{{[^"]*}}{{[/\\]+}}lib{{[/\\]+}}clang{{[/\\]+}}{{.*}}{{[/\\]+}}lib{{[/\\]+}}baremetal"
+// CHECK-V6M-DEFAULTCXX-SAME: 
"-L{{[^"]*}}{{[/\\]+}}lib{{(64)?}}{{[/\\]+}}clang{{[/\\]+}}{{.*}}{{[/\\]+}}lib{{[/\\]+}}baremetal"
 // CHECK-V6M-DEFAULTCXX-SAME: "-lc++" "-lc++abi" "-lunwind"
 // CHECK-V6M-DEFAULTCXX-SAME: "-lc" "-lm" "-lclang_rt.builtins-armv6m.a"
 // CHECK-V6M-DEFAULTCXX-SAME: "-o" "{{.*}}.o"
@@ -46,7 +46,7 @@
 // CHECK-V6M-LIBCXX-NOT: "-internal-isystem" 
"{{[^"]+}}{{[/\\]+}}include{{[/\\]+}}c++{{[/\\]+}}{{[^v].*}}"
 // CHECK-V6M-LIBCXX: "-internal-isystem" 
"{{[^"]+}}{{[/\\]+}}include{{[/\\]+}}c++{{[/\\]+}}v1"
 // CHECK-V6M-LIBCXX: "{{[^"]*}}ld.lld" "{{.*}}.o" "-Bstatic"
-// CHECK-V6M-LIBCXX-SAME: 
"-L{{[^"]*}}{{[/\\]+}}lib{{[/\\]+}}clang{{[/\\]+}}{{.*}}{{[/\\]+}}lib{{[/\\]+}}baremetal"
+// CHECK-V6M-LIBCXX-SAME: 
"-L{{[^"]*}}{{[/\\]+}}lib{{(64)?}}{{[/\\]+}}clang{{[/\\]+}}{{.*}}{{[/\\]+}}lib{{[/\\]+}}baremetal"
 // CHECK-V6M-LIBCXX-SAME: "-lc++" "-lc++abi" "-lunwind"
 // CHECK-V6M-LIBCXX-SAME: "-lc" "-lm" "-lclang_rt.builtins-armv6m.a"
 // CHECK-V6M-LIBCXX-SAME: "-o" "{{.*}}.o"
@@ -59,7 +59,7 @@
 // CHECK-V6M-LIBSTDCXX-NOT: "-internal-isystem" 
"{{[^"]+}}{{[/\\]+}}include{{[/\\]+}}c++{{[/\\]+}}v1"
 // CHECK-V6M-LIBSTDCXX: "-internal-isystem" 
"{{[^"]+}}{{[/\\]+}}include{{[/\\]+}}c++{{[/\\]+}}6.0.0"
 // CHECK-V6M-LIBSTDCXX: "{{[^"]*}}ld.lld" "{{.*}}.o" "-Bstatic"
-// CHECK-V6M-LIBSTDCXX-SAME: 
"-L{{[^"]*}}{{[/\\]+}}lib{{[/\\]+}}clang{{[/\\]+}}{{.*}}{{[/\\]+}}lib{{[/\\]+}}baremetal"
+// CHECK-V6M-LIBSTDCXX-SAME: 
"-L{{[^"]*}}{{[/\\]+}}lib{{(64)?}}{{[/\\]+}}clang{{[/\\]+}}{{.*}}{{[/\\]+}}lib{{[/\\]+}}baremetal"
 // CHECK-V6M-LIBSTDCXX-SAME: "-lstdc++" "-lsupc++" "-lunwind"
 // CHECK-V6M-LIBSTDCXX-SAME: "-lc" "-lm" "-lclang_rt.builtins-armv6m.a"
 // CHECK-V6M-LIBSTDCXX-SAME: "-o" "{{.*}}.o"
@@ -70,7 +70,7 @@
 // RUN: -nodefaultlibs \
 // RUN:   | FileCheck --check-prefix=CHECK-V6M-NDL %s
 // CHECK-V6M-NDL: "{{[^"]*}}ld.lld" "{{.*}}.o" "-Bstatic"
-// CHECK-V6M-NDL-SAME: 
"-L{{[^"]*}}{{[/\\]+}}lib{{[/\\]+}}clang{{[/\\]+}}{{.*}}{{[/\\]+}}lib{{[/\\]+}}baremetal"
 "-o" "{{.*}}.o"
+// CHECK-V6M-NDL-SAME: 
"-L{{[^"]*}}{{[/\\]+}}lib{{(64)?}}{{[/\\]+}}clang{{[/\\]+}}{{.*}}{{[/\\]+}}lib{{[/\\]+}}baremetal"
 "-o" "{{.*}}.o"
 
 // RUN: %clangxx -target arm-none-eabi -v 2>&1 \
 // RUN:   | FileCheck %s --check-prefix=CHECK-THREAD-MODEL


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libcxxabi] r305278 - Set a default value for LIBCXXABI_LIBDIR_SUFFIX, fixes installing into lib64 after r304374

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Tue Jun 13 03:16:44 2017
New Revision: 305278

URL: http://llvm.org/viewvc/llvm-project?rev=305278&view=rev
Log:
Set a default value for LIBCXXABI_LIBDIR_SUFFIX, fixes installing into lib64 
after r304374

Modified:
libcxxabi/trunk/CMakeLists.txt

Modified: libcxxabi/trunk/CMakeLists.txt
URL: 
http://llvm.org/viewvc/llvm-project/libcxxabi/trunk/CMakeLists.txt?rev=305278&r1=305277&r2=305278&view=diff
==
--- libcxxabi/trunk/CMakeLists.txt (original)
+++ libcxxabi/trunk/CMakeLists.txt Tue Jun 13 03:16:44 2017
@@ -67,6 +67,8 @@ option(LIBCXXABI_ENABLE_NEW_DELETE_DEFIN
 provides these definitions" ON)
 option(LIBCXXABI_BUILD_32_BITS "Build 32 bit libc++abi." ${LLVM_BUILD_32_BITS})
 option(LIBCXXABI_INCLUDE_TESTS "Generate build targets for the libc++abi unit 
tests." ${LLVM_INCLUDE_TESTS})
+set(LIBCXXABI_LIBDIR_SUFFIX "${LLVM_LIBDIR_SUFFIX}" CACHE STRING
+"Define suffix of library directory name (32/64)")
 set(LIBCXXABI_TARGET_TRIPLE "" CACHE STRING "Target triple for cross 
compiling.")
 set(LIBCXXABI_GCC_TOOLCHAIN "" CACHE PATH "GCC toolchain for cross compiling.")
 set(LIBCXXABI_SYSROOT "" CACHE PATH "Sysroot for cross compiling.")


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r283716 - [x86][inline-asm][clang] accept 'v' constraint

[Ismail Donmez via cfe-commits]

Hi,

On Mon, Oct 10, 2016 at 8:45 AM, Michael Zuckerman via cfe-commits
 wrote:
> Author: mzuckerm
> Date: Mon Oct 10 00:45:54 2016
> New Revision: 283716
>
> URL: http://llvm.org/viewvc/llvm-project?rev=283716&view=rev
> Log:
> [x86][inline-asm][clang] accept 'v' constraint
>
> Commit in the name of: Coby Tayree
>
> 1.'v' constraint for (x86) non-avx arch imitates the already implemented 'x' 
> constraint, i.e. allows XMM{0-15} & YMM{0-15} depending on the apparent arch 
> & mode (32/64).
> 2.for the avx512 arch it allows [X,Y,Z]MM{0-31} (mode dependent)
>
> This patch applies the needed changes to clang
>  LLVM patch: https://reviews.llvm.org/D25005
>
> Differential Revision: D25004

This fails on Linux x86-64:

 --
Exit Code: 1

Command Output (stderr):
--
/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/CodeGen/x86-inline-asm-v-constraint.c:10:9:
error: expected string not found in input
// SSE: call <4 x float> asm "vmovhlps $1, $2, $0",
"=v,v,v,~{dirflag},~{fpsr},~{flags}"(i64 %0, <4 x float> %1)
^
:1:1: note: scanning from here
 ; ModuleID = 
'/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/CodeGen/x86-inline-asm-v-constraint.c'
 ^
:14:7: note: possible intended match here
%3 = call <4 x float> asm "vmovhlps $1, $2, $0",
"=v,v,v,~{dirflag},~{fpsr},~{flags}"(i64 %1, <4 x float> %2) #1,
!srcloc !1
  ^

--
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r319322 - Fix function call to fix build

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Wed Nov 29 07:18:02 2017
New Revision: 319322

URL: http://llvm.org/viewvc/llvm-project?rev=319322&view=rev
Log:
Fix function call to fix build

../tools/clang/lib/Driver/ToolChains/Cuda.cpp:80:18: error: reference to 
non-static member function must be called; did you mean to call it with no 
arguments?
if (Distro(D.getVFS).IsDebian())
   ~~^~
   ()


Modified:
cfe/trunk/lib/Driver/ToolChains/Cuda.cpp

Modified: cfe/trunk/lib/Driver/ToolChains/Cuda.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains/Cuda.cpp?rev=319322&r1=319321&r2=319322&view=diff
==
--- cfe/trunk/lib/Driver/ToolChains/Cuda.cpp (original)
+++ cfe/trunk/lib/Driver/ToolChains/Cuda.cpp Wed Nov 29 07:18:02 2017
@@ -77,7 +77,7 @@ CudaInstallationDetector::CudaInstallati
 for (const char *Ver : Versions)
   CudaPathCandidates.push_back(D.SysRoot + "/usr/local/cuda-" + Ver);
 
-if (Distro(D.getVFS).IsDebian())
+if (Distro(D.getVFS()).IsDebian())
   // Special case for Debian to have nvidia-cuda-toolkit work
   // out of the box. More info on http://bugs.debian.org/882505
   CudaPathCandidates.push_back(D.SysRoot + "/usr/lib/cuda");


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r295433 - Update Bugzilla URLs in docs

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Fri Feb 17 02:26:54 2017
New Revision: 295433

URL: http://llvm.org/viewvc/llvm-project?rev=295433&view=rev
Log:
Update Bugzilla URLs in docs

Modified:
cfe/trunk/docs/UsersManual.rst

Modified: cfe/trunk/docs/UsersManual.rst
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/UsersManual.rst?rev=295433&r1=295432&r2=295433&view=diff
==
--- cfe/trunk/docs/UsersManual.rst (original)
+++ cfe/trunk/docs/UsersManual.rst Fri Feb 17 02:26:54 2017
@@ -1879,7 +1879,7 @@ missing from this list, please send an e
 currently excludes C++; see :ref:`C++ Language Features `. Also, this
 list does not include bugs in mostly-implemented features; please see
 the `bug
-tracker 
`_
+tracker 
`_
 for known existing bugs (FIXME: Is there a section for bug-reporting
 guidelines somewhere?).
 
@@ -2516,7 +2516,7 @@ official `MinGW-w64 website `_ on
+`Some tests might fail `_ on
 ``x86_64-w64-mingw32``.
 
 .. _clang-cl:
@@ -2559,7 +2559,7 @@ options are spelled with a leading ``/``
 
 clang-cl.exe: error: no such file or directory: '/foobar'
 
-Please `file a bug 
`_
+Please `file a bug 
`_
 for any valid cl.exe flags that clang-cl does not understand.
 
 Execute ``clang-cl /?`` to see a list of supported options:


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r300969 - Add support for openSUSE ARM Triples

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Fri Apr 21 07:26:43 2017
New Revision: 300969

URL: http://llvm.org/viewvc/llvm-project?rev=300969&view=rev
Log:
Add support for openSUSE ARM Triples

Summary: Add support for armv6hl-suse-linux-gnueabi and 
armv7hl-suse-linux-gnueabi triples used by openSUSE/SUSE.

Reviewers: cfe-commits, rovka, compnerd, rengolin

Reviewed By: rengolin

Subscribers: aemerson, rengolin, bkramer, chandlerc

Differential Revision: https://reviews.llvm.org/D32347

Added:
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/crt1.o
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/crti.o
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/crtn.o
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/gcc/

cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/gcc/armv6hl-suse-linux-gnueabi/

cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/gcc/armv6hl-suse-linux-gnueabi/5/

cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/gcc/armv6hl-suse-linux-gnueabi/5/crtbegin.o

cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/gcc/armv6hl-suse-linux-gnueabi/5/crtend.o
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/lib/
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/lib/crt1.o
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/lib/crti.o
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/lib/crtn.o
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/lib/gcc/

cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/lib/gcc/armv7hl-suse-linux-gnueabi/

cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/lib/gcc/armv7hl-suse-linux-gnueabi/5/

cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/lib/gcc/armv7hl-suse-linux-gnueabi/5/crtbegin.o

cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv7hl_tree/usr/lib/gcc/armv7hl-suse-linux-gnueabi/5/crtend.o
Modified:
cfe/trunk/lib/Driver/ToolChains/Gnu.cpp
cfe/trunk/test/Driver/linux-ld.c

Modified: cfe/trunk/lib/Driver/ToolChains/Gnu.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains/Gnu.cpp?rev=300969&r1=300968&r2=300969&view=diff
==
--- cfe/trunk/lib/Driver/ToolChains/Gnu.cpp (original)
+++ cfe/trunk/lib/Driver/ToolChains/Gnu.cpp Fri Apr 21 07:26:43 2017
@@ -1747,7 +1747,9 @@ bool Generic_GCC::GCCInstallationDetecto
   static const char *const ARMTriples[] = {"arm-linux-gnueabi",
"arm-linux-androideabi"};
   static const char *const ARMHFTriples[] = {"arm-linux-gnueabihf",
- "armv7hl-redhat-linux-gnueabi"};
+ "armv7hl-redhat-linux-gnueabi",
+ "armv6hl-suse-linux-gnueabi",
+ "armv7hl-suse-linux-gnueabi"};
   static const char *const ARMebLibDirs[] = {"/lib"};
   static const char *const ARMebTriples[] = {"armeb-linux-gnueabi",
  "armeb-linux-androideabi"};

Added: 
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/crt1.o
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/crt1.o?rev=300969&view=auto
==
(empty)

Added: 
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/crti.o
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/crti.o?rev=300969&view=auto
==
(empty)

Added: 
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/crtn.o
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/crtn.o?rev=300969&view=auto
==
(empty)

Added: 
cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/gcc/armv6hl-suse-linux-gnueabi/5/crtbegin.o
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tree/usr/lib/gcc/armv6hl-suse-linux-gnueabi/5/crtbegin.o?rev=300969&view=auto
=

r301302 - Fix fuzzer.c test on platforms where CLANG_DEFAULT_CXX_STDLIB is libc++

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Tue Apr 25 06:24:14 2017
New Revision: 301302

URL: http://llvm.org/viewvc/llvm-project?rev=301302&view=rev
Log:
Fix fuzzer.c test on platforms where CLANG_DEFAULT_CXX_STDLIB is libc++

Modified:
cfe/trunk/test/Driver/fuzzer.c

Modified: cfe/trunk/test/Driver/fuzzer.c
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/fuzzer.c?rev=301302&r1=301301&r2=301302&view=diff
==
--- cfe/trunk/test/Driver/fuzzer.c (original)
+++ cfe/trunk/test/Driver/fuzzer.c Tue Apr 25 06:24:14 2017
@@ -7,7 +7,7 @@
 // CHECK-COVERAGE-SAME: -fsanitize-coverage-indirect-calls
 // CHECK-COVERAGE-SAME: -fsanitize-coverage-trace-cmp
 
-// RUN: %clang -fsanitize=fuzzer -target i386-unknown-linux %s -### 2>&1 | 
FileCheck --check-prefixes=CHECK-LIBCXX-LINUX %s
+// RUN: %clang -fsanitize=fuzzer -target i386-unknown-linux -stdlib=platform 
%s -### 2>&1 | FileCheck --check-prefixes=CHECK-LIBCXX-LINUX %s
 //
 // CHECK-LIBCXX-LINUX: -lstdc++
 


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [libcxx] r285382 - Add __libcpp_version file and __libcpp_library_version function.

[Ismail Donmez via cfe-commits]

Hi,

On Fri, Oct 28, 2016 at 9:06 AM, Eric Fiselier via cfe-commits
 wrote:
> Author: ericwf
> Date: Fri Oct 28 01:06:50 2016
> New Revision: 285382
>
> URL: http://llvm.org/viewvc/llvm-project?rev=285382&view=rev
> Log:
> Add __libcpp_version file and __libcpp_library_version function.
>
> This patch does two seperate things. First it adds a file called
> "__libcpp_version" which only contains the current libc++ version
> (currently 4000). This file is not intended for use as a header. This file
> is used by Clang in order to easily determine the installed libc++ version.
> This allows Clang to enable/disable certain language features only when the
> library supports them.
>
> The second change is the addition of _LIBCPP_LIBRARY_VERSION macro, which
> returns the version of the installed dylib since it may be different than
> the headers.

The test seems to be failing here:

[ 7087s]  TEST 'libc++ :: libcxx/version.pass.cpp'
FAILED 
[ 7087s] Compiled With:
['/home/abuild/rpmbuild/BUILD/llvm/stage1/bin/clang++', '-o',
'/home/abuild/rpmbuild/BUILD/llvm/stage2/projects/libcxx/test/libcxx/Output/version.pass.cpp.o',
'-x', 'c++', 
'/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/libcxx/version.pass.cpp',
'-c', '-v', '-Werror=thread-safety', '-std=c++1z', '-include',
'/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/support/nasty_macros.hpp',
'-nostdinc++', '-I/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/include',
'-D__STDC_FORMAT_MACROS', '-D__STDC_LIMIT_MACROS',
'-D__STDC_CONSTANT_MACROS',
'-I/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/support',
'-DLIBCXX_FILESYSTEM_STATIC_TEST_ROOT="/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/experimental/filesystem/Inputs/static_test_env"',
'-DLIBCXX_FILESYSTEM_DYNAMIC_TEST_ROOT="/home/abuild/rpmbuild/BUILD/llvm/stage2/projects/libcxx/test/filesystem/Output/dynamic_env"',
'-DLIBCXX_FILESYSTEM_DYNAMIC_TEST_HELPER="/usr/bin/python2.7
/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/support/filesystem_dynamic_test_helper.py"',
'-c', '&&', '/home/abuild/rpmbuild/BUILD/llvm/stage1/bin/clang++',
'-o', 
'/home/abuild/rpmbuild/BUILD/llvm/stage2/projects/libcxx/test/libcxx/Output/version.pass.cpp.exe',
'/home/abuild/rpmbuild/BUILD/llvm/stage2/projects/libcxx/test/libcxx/Output/version.pass.cpp.o',
'-v', '-L/home/abuild/rpmbuild/BUILD/llvm/stage2/lib64',
'-Wl,-rpath,/home/abuild/rpmbuild/BUILD/llvm/stage2/lib64',
'-nodefaultlibs', '-lc++experimental', '-lc++', '-lm', '-lgcc_s',
'-lgcc', '-lpthread', '-lc', '-lgcc_s', '-lgcc']
[ 7087s] Command: ['env',
'LIBCXX_FILESYSTEM_DYNAMIC_TEST_ROOT=/home/abuild/rpmbuild/BUILD/llvm/stage2/projects/libcxx/test/filesystem/Output/dynamic_env',
'/home/abuild/rpmbuild/BUILD/llvm/stage2/projects/libcxx/test/libcxx/Output/version.pass.cpp.exe']
[ 7087s] Exit Code: -6
[ 7087s] Standard Error:
[ 7087s] --
[ 7087s] version.pass.cpp.exe:
/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/libcxx/version.pass.cpp:26:
int main(): Assertion `_LIBCPP_VERSION == _LIBCPP_LIBRARY_VERSION'
failed.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r247668 - Doxygen fix InitializeBuiltins -> initializeBuiltins

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Tue Sep 15 04:53:14 2015
New Revision: 247668

URL: http://llvm.org/viewvc/llvm-project?rev=247668&view=rev
Log:
Doxygen fix InitializeBuiltins -> initializeBuiltins


Modified:
cfe/trunk/lib/Basic/Builtins.cpp

Modified: cfe/trunk/lib/Basic/Builtins.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Builtins.cpp?rev=247668&r1=247667&r2=247668&view=diff
==
--- cfe/trunk/lib/Basic/Builtins.cpp (original)
+++ cfe/trunk/lib/Basic/Builtins.cpp Tue Sep 15 04:53:14 2015
@@ -62,7 +62,7 @@ bool Builtin::Context::builtinIsSupporte
  !GnuModeUnsupported && !MSModeUnsupported && !ObjCUnsupported;
 }
 
-/// InitializeBuiltins - Mark the identifiers for all the builtins with their
+/// initializeBuiltins - Mark the identifiers for all the builtins with their
 /// appropriate builtin ID # and mark any non-portable builtin identifiers as
 /// such.
 void Builtin::Context::initializeBuiltins(IdentifierTable &Table,


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D13549: Added new options to ClangFormat VSIX package.

[Ismail Donmez via cfe-commits]

On Mon, Oct 19, 2015 at 7:45 PM, Hans Wennborg via cfe-commits <
cfe-commits@lists.llvm.org> wrote:

> hans added a comment.
>
> This is now part of the latest snapshot at http://llvm.org/builds/
> Seems to work :-)
>
> I had to dig around a bit to figure out where these settings are actually
> exposed. Should we mention that in the documentation somewhere? Actually,
> do we even have any documentation for this plugin?
>
>
Options are exposed in Tools->Options->LLVM/Clang->ClangFormat it seems
(which is very nice btw!)

ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r252836 - [CMake] Setup an install component for libclang and c-index-test.

[Ismail Donmez via cfe-commits]

Hi,

On Thu, Nov 12, 2015 at 2:46 AM, Argyrios Kyrtzidis via cfe-commits <
cfe-commits@lists.llvm.org> wrote:

> --- cfe/trunk/tools/c-index-test/CMakeLists.txt (original)
> +++ cfe/trunk/tools/c-index-test/CMakeLists.txt Wed Nov 11 18:46:57 2015
> @@ -28,3 +28,12 @@ if (CLANG_HAVE_LIBXML)
>include_directories(SYSTEM ${LIBXML2_INCLUDE_DIR})
>target_link_libraries(c-index-test ${LIBXML2_LIBRARIES})
>  endif()
> +
> +install(TARGETS c-index-test
> +  RUNTIME DESTINATION local/bin
> +  COMPONENT c-index-test)
>

This doesn't look right, all of the llvm/clang install binaries under
$PREFIX/bin, with this change c-index-test is going under $PREFIX/local/bin

ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r252890 - Fix c-index-test install path

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Thu Nov 12 07:47:35 2015
New Revision: 252890

URL: http://llvm.org/viewvc/llvm-project?rev=252890&view=rev
Log:
Fix c-index-test install path

Modified:
cfe/trunk/tools/c-index-test/CMakeLists.txt

Modified: cfe/trunk/tools/c-index-test/CMakeLists.txt
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/c-index-test/CMakeLists.txt?rev=252890&r1=252889&r2=252890&view=diff
==
--- cfe/trunk/tools/c-index-test/CMakeLists.txt (original)
+++ cfe/trunk/tools/c-index-test/CMakeLists.txt Thu Nov 12 07:47:35 2015
@@ -30,7 +30,7 @@ if (CLANG_HAVE_LIBXML)
 endif()
 
 install(TARGETS c-index-test
-  RUNTIME DESTINATION local/bin
+  RUNTIME DESTINATION bin
   COMPONENT c-index-test)
 add_custom_target(install-c-index-test
   DEPENDS c-index-test


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r252836 - [CMake] Setup an install component for libclang and c-index-test.

[Ismail Donmez via cfe-commits]

Fixed in r252890

On Thu, Nov 12, 2015 at 2:17 PM, Ismail Donmez  wrote:

> Hi,
>
> On Thu, Nov 12, 2015 at 2:46 AM, Argyrios Kyrtzidis via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> --- cfe/trunk/tools/c-index-test/CMakeLists.txt (original)
>> +++ cfe/trunk/tools/c-index-test/CMakeLists.txt Wed Nov 11 18:46:57 2015
>> @@ -28,3 +28,12 @@ if (CLANG_HAVE_LIBXML)
>>include_directories(SYSTEM ${LIBXML2_INCLUDE_DIR})
>>target_link_libraries(c-index-test ${LIBXML2_LIBRARIES})
>>  endif()
>> +
>> +install(TARGETS c-index-test
>> +  RUNTIME DESTINATION local/bin
>> +  COMPONENT c-index-test)
>>
>
> This doesn't look right, all of the llvm/clang install binaries under
> $PREFIX/bin, with this change c-index-test is going under $PREFIX/local/bin
>
> ismail
>
>
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r278140 - [CUDA] Regression test to make sure C++ include path are forwarded to host and device frontends.

[Ismail Donmez via cfe-commits]

Hi,

On Tue, Aug 9, 2016 at 8:27 PM, Samuel Antao via cfe-commits
 wrote:
> Author: sfantao
> Date: Tue Aug  9 12:27:24 2016
> New Revision: 278140
>
> URL: http://llvm.org/viewvc/llvm-project?rev=278140&view=rev
> Log:
> [CUDA] Regression test to make sure C++ include path are forwarded to host 
> and device frontends.
>
> Summary: Add test to detect the C++ include paths are passed to both CUDA 
> host and device frontends.
>
> Reviewers: tra
>
> Subscribers: cfe-commits
>
> Differential Revision: https://reviews.llvm.org/D22946
>
> Modified:
> cfe/trunk/test/Driver/cuda-detect.cu
>
> Modified: cfe/trunk/test/Driver/cuda-detect.cu
> URL: 
> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cuda-detect.cu?rev=278140&r1=278139&r2=278140&view=diff
> ==
> --- cfe/trunk/test/Driver/cuda-detect.cu (original)
> +++ cfe/trunk/test/Driver/cuda-detect.cu Tue Aug  9 12:27:24 2016
> @@ -72,6 +72,11 @@
>  // RUN:   | FileCheck %s -check-prefix COMMON \
>  // RUN: -check-prefix NOCUDAINC -check-prefix NOLIBDEVICE
>
> +// Verify that C++ include paths are passed for both host and device 
> frontends.
> +// RUN: %clang -### -target x86_64-linux-gnu %s \
> +// RUN: --sysroot=%S/Inputs/ubuntu_14.04_multiarch_tree2 2>&1 \
> +// RUN: | FileCheck %s --check-prefix CHECK-CXXINCLUDE
> +
>  // CHECK: Found CUDA installation: {{.*}}/Inputs/CUDA/usr/local/cuda
>  // NOCUDA-NOT: Found CUDA installation:
>
> @@ -92,3 +97,8 @@
>  // CUDAINC-SAME: "-include" "__clang_cuda_runtime_wrapper.h"
>  // NOCUDAINC-NOT: "-include" "__clang_cuda_runtime_wrapper.h"
>  // COMMON-SAME: "-x" "cuda"
> +// CHECK-CXXINCLUDE: clang{{.*}} "-cc1" "-triple" "nvptx64-nvidia-cuda"
> +// CHECK-CXXINCLUDE-SAME: {{.*}}"-internal-isystem" "{{.+}}/include/c++/4.8"
> +// CHECK-CXXINCLUDE: clang{{.*}} "-cc1" "-triple" "x86_64--linux-gnu"
> +// CHECK-CXXINCLUDE-SAME: {{.*}}"-internal-isystem" "{{.+}}/include/c++/4.8"
> +// CHECK-CXXINCLUDE: ld{{.*}}"

This needs -stdlib=libstdc++ otherwise it fails on hosts defaulting to libc++.

ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [libcxx] r260337 - Fix overload sets of strchr, strpbrk, strrchr, memchr and strstr from

[Ismail Donmez via cfe-commits]

Hi,

On Wed, Feb 10, 2016 at 2:59 AM, Richard Smith via cfe-commits
 wrote:
> Author: rsmith
> Date: Tue Feb  9 18:59:02 2016
> New Revision: 260337
>
> URL: http://llvm.org/viewvc/llvm-project?rev=260337&view=rev
> Log:
> Fix overload sets of strchr, strpbrk, strrchr, memchr and strstr from
>  and wcschr, wcspbrk, wcsrchr, wmemchr, and wcsstr from  to
> provide a const-correct overload set even when the underlying C library does
> not.
>
> This change adds a new macro, _LIBCPP_PREFERRED_OVERLOAD, which (if defined)
> specifies that a given overload is a better match than an otherwise equally
> good function declaration without the overload. This is implemented in modern
> versions of Clang via __attribute__((enable_if)), and not elsewhere.
>
> We use this new macro to define overloads in the global namespace for these
> functions that displace the overloads provided by the C library, unless we
> believe the C library is already providing the correct signatures.
>
> Added:
> libcxx/trunk/include/string.h
>   - copied, changed from r251642, libcxx/trunk/include/string.h
> Modified:
> libcxx/trunk/include/__config
> libcxx/trunk/include/cstring
> libcxx/trunk/include/cwchar
> libcxx/trunk/include/wchar.h
> libcxx/trunk/test/std/depr/depr.c.headers/string_h.pass.cpp
> libcxx/trunk/test/std/depr/depr.c.headers/wchar_h.pass.cpp
> libcxx/trunk/test/std/strings/c.strings/cstring.pass.cpp
> libcxx/trunk/test/std/strings/c.strings/cwchar.pass.cpp

Fails on Linux x86-64 with glibc 2.22:

/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/depr/depr.c.headers/string_h.pass.cpp:55:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^ 
/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/depr/depr.c.headers/string_h.pass.cpp:56:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^
~~~
/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/depr/depr.c.headers/string_h.pass.cpp:57:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^ ~
/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/depr/depr.c.headers/string_h.pass.cpp:58:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^ ~~
/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/depr/depr.c.headers/string_h.pass.cpp:59:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^
~~~
5 errors generated.


/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/strings/c.strings/cstring.pass.cpp:55:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^

/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/strings/c.strings/cstring.pass.cpp:56:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^
~
/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/strings/c.strings/cstring.pass.cpp:57:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^

/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/strings/c.strings/cstring.pass.cpp:58:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^
~~
/home/abuild/rpmbuild/BUILD/llvm/projects/libcxx/test/std/strings/c.strings/cstring.pass.cpp:59:5:
error: static_assert failed ""
static_assert((std::is_same::value), "");
^
~~~
5 errors generated.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r261297 - Implement the likely resolution of core issue 253.

[Ismail Donmez via cfe-commits]

Hi,

On Fri, Feb 19, 2016 at 3:52 AM, Nico Weber via cfe-commits
 wrote:
> Author: nico
> Date: Thu Feb 18 19:52:46 2016
> New Revision: 261297
>
> URL: http://llvm.org/viewvc/llvm-project?rev=261297&view=rev
> Log:
> Implement the likely resolution of core issue 253.
>
> C++11 requires const objects to have a user-provided constructor, even for
> classes without any fields. DR 253 relaxes this to say "If the implicit 
> default
> constructor initializes all subobjects, no initializer should be required."
>
> clang is currently the only compiler that implements this C++11 rule, and e.g.
> libstdc++ relies on something like DR 253 to compile in newer versions.  This
> change  makes it possible to build code that says `const vector v;' again
> when using libstdc++5.2 and _GLIBCXX_DEBUG
> (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60284).
>
> Fixes PR23381.

Since nearly all Linux distros now ship gcc5, it would be nice to have
this fix in 3.8 release.

Thanks,
ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r266108 - clang-cl: Remove -isystem, add -imsvc.

[Ismail Donmez via cfe-commits]

Hi,

On Tue, Apr 12, 2016 at 10:04 PM, Nico Weber via cfe-commits
 wrote:
> Author: nico
> Date: Tue Apr 12 14:04:37 2016
> New Revision: 266108
>
> URL: http://llvm.org/viewvc/llvm-project?rev=266108&view=rev
> Log:
> clang-cl: Remove -isystem, add -imsvc.
>
> r260990 exposed -isystem in clang-cl. -isystem adds a directory to the front 
> of
> the system include search path. The idea was to use this to point to a 
> hermetic
> msvc install, but as it turns out this doesn't work: -isystem then adds the
> hermetic headers in front of clang's builtin headers, and clang's headers that
> are supposed to wrap msvc headers (say, stdarg.h) aren't picked up at all
> anymore.
>
> So revert that, and instead expose -imsvc which works as if the passed
> directory was part of %INCLUDE%: The header is treated as a system header, but
> it is searched after clang's lib/Header headers.
>
> Fixes half of PRPR26751.

This fails the test for me on Linux x86-64:

 /home/abuild/rpmbuild/BUILD/llvm/stage2/./bin/clang --driver-mode=cl
 --driver-mode=cl  -fcolor-diagnostics
-fno-color-diagnostics  -fdiagnostics-color
-fno-diagnostics-color  -fdiagnostics-parseable-fixits
-ferror-limit=10  -fmsc-version=1800  -fno-strict-aliasing
 -fstrict-aliasing  -fsyntax-only  -fms-compatibility
-fno-ms-compatibility  -fms-extensions  -fno-ms-extensions
 -mllvm -disable-llvm-optzns  -Wunused-variable
-fmacro-backtrace-limit=0  -Werror /Zs --
/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/cl-options.c
2>&1
 --
 Exit Code: 1

 Command Output (stderr):
 --
 /home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/cl-options.c:88:17:
error: expected string not found in input
 // SLASH_imsvc: "-internal-isystem"
"{{[^"]*}}lib{{/|}}clang{{[^"]*}}include"
 ^
 :1:1: note: scanning from here
 openSUSE Linux clang version 3.9.0 (trunk 266277) (based on LLVM 3.9.0svn)
 ^
 :5:709: note: possible intended match here
  "/home/abuild/rpmbuild/BUILD/llvm/stage2/bin/clang-3.9" "-cc1"
"-triple" "x86_64-pc-windows-msvc18.0.0" "-emit-obj" "-mrelax-all"
"-mincremental-linker-compatible" "-disable-free"
"-disable-llvm-verifier" "-discard-value-names" "-main-file-name"
"cl-options.c" "-mrelocation-model" "pic" "-pic-level" "2"
"-mthread-model" "posix" "-relaxed-aliasing" "-fmath-errno"
"-masm-verbose" "-mconstructor-aliases" "-munwind-tables"
"-target-cpu" "x86-64" "-D_MT" "--dependent-lib=libcmt"
"--dependent-lib=oldnames" "-fms-volatile" "-fdiagnostics-format"
"msvc" "-momit-leaf-frame-pointer" "-dwarf-column-info"
"-debugger-tuning=gdb" "-resource-dir"
"/home/abuild/rpmbuild/BUILD/llvm/stage2/bin/../lib64/clang/3.9.0"
"-internal-isystem"
"/home/abuild/rpmbuild/BUILD/llvm/stage2/bin/../lib64/clang/3.9.0/include"
"-internal-isystem" "myincludedir" "-internal-isystem" "C:/Program
Files/Microsoft Visual Studio 10.0/VC/include" "-internal-isystem"
"C:/Program Files/Microsoft Visual Studio 9.0/VC/include"
"-internal-isystem" "C:/Program Files/Microsoft Visual Studio
9.0/VC/PlatformSDK/Include" "-internal-isystem" "C:/Program
Files/Microsoft Visual Studio 8/VC/include" "-internal-isystem"
"C:/Program Files/Microsoft Visual Studio 8/VC/PlatformSDK/Include"
"-fdebug-compilation-dir"
"/home/abuild/rpmbuild/BUILD/llvm/stage2/tools/clang/test/Driver"
"-ferror-limit" "19" "-fmessage-length" "0" "-fms-extensions"
"-fms-compatibility" "-fms-compatibility-version=18"
"-fno-threadsafe-statics" "-fdelayed-template-parsing"
"-fobjc-runtime=gcc" "-fdiagnostics-show-option" "-o"
"/tmp/cl-options-ca1c04.obj" "-x" "c"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/cl-options.c"

Any ideas?
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r266108 - clang-cl: Remove -isystem, add -imsvc.

[Ismail Donmez via cfe-commits]

On Thu, Apr 14, 2016 at 2:09 PM, Nico Weber  wrote:
> It's supposed to match the
>
> "-internal-isystem"
> "/home/abuild/rpmbuild/BUILD/llvm/stage2/bin/../lib64/clang/3.9.0/include"
> "-internal-isystem" "myincludedir"
>
> bit in your output. Looks like it's "lib64" instead of "lib" on your system.
> Does changing
>
> // SLASH_imsvc: "-internal-isystem"
> "{{[^"]*}}lib{{/|}}clang{{[^"]*}}include"
>
> to
>
> // SLASH_imsvc: "-internal-isystem"
> "{{[^"]*}}lib{{(64)?/|}}clang{{[^"]*}}include"
>
> help?

Yes it does, thanks.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r266324 - Fix testcase for the LLVM_LIBDIR_SUFFIX=64 case. Fallout from r266108.

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Thu Apr 14 10:32:24 2016
New Revision: 266324

URL: http://llvm.org/viewvc/llvm-project?rev=266324&view=rev
Log:
Fix testcase for the LLVM_LIBDIR_SUFFIX=64 case. Fallout from r266108.


Modified:
cfe/trunk/test/Driver/cl-options.c

Modified: cfe/trunk/test/Driver/cl-options.c
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl-options.c?rev=266324&r1=266323&r2=266324&view=diff
==
--- cfe/trunk/test/Driver/cl-options.c (original)
+++ cfe/trunk/test/Driver/cl-options.c Thu Apr 14 10:32:24 2016
@@ -85,7 +85,7 @@
 // RUN: %clang_cl /imsvcmyincludedir -### -- %s 2>&1 | FileCheck 
-check-prefix=SLASH_imsvc %s
 // RUN: %clang_cl /imsvc myincludedir -### -- %s 2>&1 | FileCheck 
-check-prefix=SLASH_imsvc %s
 // Clang's resource header directory should be first:
-// SLASH_imsvc: "-internal-isystem" 
"{{[^"]*}}lib{{/|}}clang{{[^"]*}}include"
+// SLASH_imsvc: "-internal-isystem" 
"{{[^"]*}}lib{{(64)?/|}}clang{{[^"]*}}include"
 // SLASH_imsvc: "-internal-isystem" "myincludedir"
 
 // RUN: %clang_cl /J -### -- %s 2>&1 | FileCheck -check-prefix=J %s


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r266108 - clang-cl: Remove -isystem, add -imsvc.

[Ismail Donmez via cfe-commits]

On Thu, Apr 14, 2016 at 5:52 PM, Nico Weber  wrote:
> On Thu, Apr 14, 2016 at 10:32 AM, Ismail Donmez  wrote:
>>
>> On Thu, Apr 14, 2016 at 2:09 PM, Nico Weber  wrote:
>> > It's supposed to match the
>> >
>> > "-internal-isystem"
>> >
>> > "/home/abuild/rpmbuild/BUILD/llvm/stage2/bin/../lib64/clang/3.9.0/include"
>> > "-internal-isystem" "myincludedir"
>> >
>> > bit in your output. Looks like it's "lib64" instead of "lib" on your
>> > system.
>> > Does changing
>> >
>> > // SLASH_imsvc: "-internal-isystem"
>> > "{{[^"]*}}lib{{/|}}clang{{[^"]*}}include"
>> >
>> > to
>> >
>> > // SLASH_imsvc: "-internal-isystem"
>> > "{{[^"]*}}lib{{(64)?/|}}clang{{[^"]*}}include"
>> >
>> > help?
>>
>> Yes it does, thanks.
>
>
> Want to land that change? :-)

r266324 thanks! :)
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r270373 - [driver][mips] Support new versions of MIPS CodeScape toolchains

[Ismail Donmez via cfe-commits]

Hi,

On Sun, May 22, 2016 at 9:18 PM, Simon Atanasyan via cfe-commits
 wrote:
> Author: atanasyan
> Date: Sun May 22 13:18:07 2016
> New Revision: 270373
>
> URL: http://llvm.org/viewvc/llvm-project?rev=270373&view=rev
> Log:
> [driver][mips] Support new versions of MIPS CodeScape toolchains

Two tests fail for me on Linux, please the attached build log.

Thanks,
ismail


build.txt.gz
Description: GNU Zip compressed data
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r270373 - [driver][mips] Support new versions of MIPS CodeScape toolchains

[Ismail Donmez via cfe-commits]

On Tue, May 24, 2016 at 4:18 PM, Simon Atanasyan  wrote:
> /home/abuild/rpmbuild/BUILD/llvm/stage2/bin/clang -no-canonical-prefixes \
>   /home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/mips-img-v2.cpp \
>   -### -o 
> /home/abuild/rpmbuild/BUILD/llvm/stage2/tools/clang/test/Driver/Output/mips-img-v2.cpp.tmp.o
> \
>   --target=mips-img-linux-gnu \
>   
> --gcc-toolchain=/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree
> \
>   -EB -mips32r6 -mhard-float -mabi=32

Here is the output:

openSUSE Linux clang version 3.9.0 (trunk 270535) (based on LLVM 3.9.0svn)
Target: mips-img-linux-gnu
Thread model: posix
InstalledDir: /home/abuild/rpmbuild/BUILD/llvm/stage2/bin
 "/home/abuild/rpmbuild/BUILD/llvm/stage2/bin/clang" "-cc1" "-triple"
"mips-img-linux-gnu" "-emit-obj" "-mrelax-all" "-disable-free"
"-disable-llvm-verifier" "-discard-value-names" "-main-file-name"
"mips-img-v2.cpp" "-mrelocation-model" "static" "-mthread-model"
"posix" "-mdisable-fp-elim" "-fmath-errno" "-masm-verbose"
"-mconstructor-aliases" "-fuse-init-array" "-target-cpu" "mips32r6"
"-target-abi" "o32" "-mfloat-abi" "hard" "-dwarf-column-info"
"-debugger-tuning=gdb" "-resource-dir"
"/home/abuild/rpmbuild/BUILD/llvm/stage2/bin/../lib64/clang/3.9.0"
"-internal-isystem"
"/home/abuild/rpmbuild/BUILD/llvm/stage2/bin/../include/c++/v1"
"-internal-isystem"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/usr/local/include"
"-internal-isystem"
"/home/abuild/rpmbuild/BUILD/llvm/stage2/bin/../lib64/clang/3.9.0/include"
"-internal-externc-isystem"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/lib/../usr/include"
"-internal-externc-isystem"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/include"
"-internal-externc-isystem"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/usr/include"
"-fdeprecated-macro" "-fdebug-compilation-dir" "/home/abuild"
"-ferror-limit" "19" "-fmessage-length" "130" "-fobjc-runtime=gcc"
"-fcxx-exceptions" "-fexceptions" "-fdiagnostics-show-option" "-o"
"/tmp/mips-img-v2-677094.o" "-x" "c++"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/mips-img-v2.cpp"
 "/usr/bin/ld" "-z" "relro"
"--sysroot=/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard"
"--build-id" "--enable-new-dtags" "--eh-frame-hdr" "-m" "elf32btsmip"
"-dynamic-linker" "/lib/ld-linux-mipsn8.so.1" "-o"
"/home/abuild/rpmbuild/BUILD/llvm/stage2/tools/clang/test/Driver/Output/mips-img-v2.cpp.tmp.o"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/usr/lib/../lib/crt1.o"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/usr/lib/../lib/crti.o"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/mips-r6-hard/lib/crtbegin.o"
"-L/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-linux-gnu/lib/mips-r6-hard/lib"
"-L/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/mips-r6-hard/lib"
"-L/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-linux-gnu/lib/../lib/mips-r6-hard"
"-L/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/lib/../lib"
"-L/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/usr/lib/../lib"
"-L/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-linux-gnu/lib/mips-r6-hard"
"-L/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/lib"
"-L/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../sysroot/mips-r6-hard/usr/lib"
"/tmp/mips-img-v2-677094.o" "-lgcc" "--as-needed" "-lgcc_s"
"--no-as-needed" "-lc" "-lgcc" "--as-needed" "-lgcc_s"
"--no-as-needed"
"/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/mips-r6-hard/lib/crtend.o"
"/home/abuild/rpmbuild/BUILD/llvm/to

Re: r270373 - [driver][mips] Support new versions of MIPS CodeScape toolchains

[Ismail Donmez via cfe-commits]

Hi,

On Wed, May 25, 2016 at 1:32 PM, Simon Atanasyan  wrote:
> Hi,
>
> Thanks for the information. One more question. Does the following folder 
> exist?
>
> /home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-linux-gnu/include/c++/4.9.2
>

Yes :

:/ # ls 
/home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-linux-gnu/include/c++/4.9.2
mips-img-linux-gnu
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r270373 - [driver][mips] Support new versions of MIPS CodeScape toolchains

[Ismail Donmez via cfe-commits]

Any ideas what's going on?

On Wed, May 25, 2016 at 1:35 PM, Ismail Donmez  wrote:
> Hi,
>
> On Wed, May 25, 2016 at 1:32 PM, Simon Atanasyan  wrote:
>> Hi,
>>
>> Thanks for the information. One more question. Does the following folder 
>> exist?
>>
>> /home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-linux-gnu/include/c++/4.9.2
>>
>
> Yes :
>
> :/ # ls 
> /home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-linux-gnu/include/c++/4.9.2
> mips-img-linux-gnu
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r270373 - [driver][mips] Support new versions of MIPS CodeScape toolchains

[Ismail Donmez via cfe-commits]

Hi,

I think this is because I enable libc++ by default with
-DLLVM_ENABLE_LIBCXX=ON , so this tests need an explicit
-stdlib=libstdc++

ismail


On Thu, May 26, 2016 at 10:08 AM, Simon Atanasyan  wrote:
> I'm trying to reproduce the problem but without success yet. And all
> buildbots available to me are green. So no ideas right now.
>
> By the way, as far as I can see you use some sort of build system
> probably to get rpm package. Right? Could you try to build Clang by
> hand (http://clang.llvm.org/get_started.html) and run the tests. Is
> the problem reproduced in that case?
>
> On Thu, May 26, 2016 at 10:01 AM, Ismail Donmez  wrote:
>> Any ideas what's going on?
>>
>> On Wed, May 25, 2016 at 1:35 PM, Ismail Donmez  wrote:
>>> Hi,
>>>
>>> On Wed, May 25, 2016 at 1:32 PM, Simon Atanasyan  
>>> wrote:
 Hi,

 Thanks for the information. One more question. Does the following folder 
 exist?

 /home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-linux-gnu/include/c++/4.9.2

>>>
>>> Yes :
>>>
>>> :/ # ls 
>>> /home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-linux-gnu/include/c++/4.9.2
>>> mips-img-linux-gnu
>
> --
> Simon
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r270373 - [driver][mips] Support new versions of MIPS CodeScape toolchains

[Ismail Donmez via cfe-commits]

Indeed the problematic option is -DCLANG_DEFAULT_CXX_STDLIB=libc++ ,
thanks Jonas!

On Thu, May 26, 2016 at 11:19 AM, Hahnfeld, Jonas
 wrote:
> Hi,
>
> Have you also set CLANG_DEFAULT_CXX_STDLIB? Then you may need to add an
> '-stdlib=platform' to the test... (see r260662 and r263434)
>
> Cheers,
> Jonas
>
>> -Original Message-
>> From: cfe-commits [mailto:cfe-commits-boun...@lists.llvm.org] On Behalf
>> Of Ismail Donmez via cfe-commits
>> Sent: Thursday, May 26, 2016 10:03 AM
>> To: Simon Atanasyan
>> Cc: cfe-commits
>> Subject: Re: r270373 - [driver][mips] Support new versions of MIPS
>> CodeScape toolchains
>>
>> Hi,
>>
>> I think this is because I enable libc++ by default with -
>> DLLVM_ENABLE_LIBCXX=ON , so this tests need an explicit -stdlib=libstdc++
>>
>> ismail
>>
>>
>> On Thu, May 26, 2016 at 10:08 AM, Simon Atanasyan
>>  wrote:
>> > I'm trying to reproduce the problem but without success yet. And all
>> > buildbots available to me are green. So no ideas right now.
>> >
>> > By the way, as far as I can see you use some sort of build system
>> > probably to get rpm package. Right? Could you try to build Clang by
>> > hand (http://clang.llvm.org/get_started.html) and run the tests. Is
>> > the problem reproduced in that case?
>> >
>> > On Thu, May 26, 2016 at 10:01 AM, Ismail Donmez 
>> wrote:
>> >> Any ideas what's going on?
>> >>
>> >> On Wed, May 25, 2016 at 1:35 PM, Ismail Donmez 
>> wrote:
>> >>> Hi,
>> >>>
>> >>> On Wed, May 25, 2016 at 1:32 PM, Simon Atanasyan
>>  wrote:
>> >>>> Hi,
>> >>>>
>> >>>> Thanks for the information. One more question. Does the following
>> folder exist?
>> >>>>
>> >>>>
>> /home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mip
>> >>>> s_img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img
>> >>>> -linux-gnu/include/c++/4.9.2
>> >>>>
>> >>>
>> >>> Yes :
>> >>>
>> >>> :/ # ls
>> >>>
>> /home/abuild/rpmbuild/BUILD/llvm/tools/clang/test/Driver/Inputs/mips
>> >>> _img_v2_tree/lib/gcc/mips-img-linux-gnu/4.9.2/../../../../mips-img-l
>> >>> inux-gnu/include/c++/4.9.2
>> >>> mips-img-linux-gnu
>> >
>> > --
>> > Simon
>> ___
>> cfe-commits mailing list
>> cfe-commits@lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r270373 - [driver][mips] Support new versions of MIPS CodeScape toolchains

[Ismail Donmez via cfe-commits]

It does, thanks!

On Thu, May 26, 2016 at 3:35 PM, Simon Atanasyan  wrote:
> I hope r270842 fixes the problem.
>
> Thanks a lot for your help.
>
> On Thu, May 26, 2016 at 11:28 AM, Ismail Donmez  wrote:
>> Indeed the problematic option is -DCLANG_DEFAULT_CXX_STDLIB=libc++ ,
>> thanks Jonas!
>>
>> On Thu, May 26, 2016 at 11:19 AM, Hahnfeld, Jonas
>>  wrote:
>>> Hi,
>>>
>>> Have you also set CLANG_DEFAULT_CXX_STDLIB? Then you may need to add an
>>> '-stdlib=platform' to the test... (see r260662 and r263434)
>>>
>>> Cheers,
>>> Jonas
>>>
>>>> -----Original Message-
>>>> From: cfe-commits [mailto:cfe-commits-boun...@lists.llvm.org] On Behalf
>>>> Of Ismail Donmez via cfe-commits
>>>> Sent: Thursday, May 26, 2016 10:03 AM
>>>> To: Simon Atanasyan
>>>> Cc: cfe-commits
>>>> Subject: Re: r270373 - [driver][mips] Support new versions of MIPS
>>>> CodeScape toolchains
>>>>
>>>> Hi,
>>>>
>>>> I think this is because I enable libc++ by default with -
>>>> DLLVM_ENABLE_LIBCXX=ON , so this tests need an explicit -stdlib=libstdc++
>
> --
> Simon Atanasyan
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r271427 - [driver][arm] add armv7 and thumb include and lib paths

[Ismail Donmez via cfe-commits]

Hi,

On Wed, Jun 1, 2016 at 11:48 PM, Chih-Hung Hsieh via cfe-commits
 wrote:
> Author: chh
> Date: Wed Jun  1 15:48:46 2016
> New Revision: 271427
>
> URL: http://llvm.org/viewvc/llvm-project?rev=271427&view=rev
> Log:
> [driver][arm] add armv7 and thumb include and lib paths
>
> Add a new test android-ndk-standalone.cpp
> with new Android NDK release tree structure.
> Detect armv7 sub architecture and thumb mode,
> to add system include and link search paths.
>
> Differential Revision: http://reviews.llvm.org/D20600
>

android-ndk-standalone.cpp needs -stdlib=libstdc++ to fix test on
libc++ default systems.

ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r271427 - [driver][arm] add armv7 and thumb include and lib paths

[Ismail Donmez via cfe-commits]

This is due to lib vs lib64. Will fix. Sorry for the wrong diagnosis :)

On Thu, Jun 2, 2016 at 8:24 PM, Chih-hung Hsieh  wrote:
> Ismail,
> I saw that android-ndk-standalone.cpp has already -stdlib=libstdc++.
> Could you tell me or just go ahead and make the necessary change?
> Thanks.
>
>
> On Thu, Jun 2, 2016 at 9:52 AM, Ismail Donmez  wrote:
>>
>> Hi,
>>
>> On Wed, Jun 1, 2016 at 11:48 PM, Chih-Hung Hsieh via cfe-commits
>>  wrote:
>> > Author: chh
>> > Date: Wed Jun  1 15:48:46 2016
>> > New Revision: 271427
>> >
>> > URL: http://llvm.org/viewvc/llvm-project?rev=271427&view=rev
>> > Log:
>> > [driver][arm] add armv7 and thumb include and lib paths
>> >
>> > Add a new test android-ndk-standalone.cpp
>> > with new Android NDK release tree structure.
>> > Detect armv7 sub architecture and thumb mode,
>> > to add system include and link search paths.
>> >
>> > Differential Revision: http://reviews.llvm.org/D20600
>> >
>>
>> android-ndk-standalone.cpp needs -stdlib=libstdc++ to fix test on
>> libc++ default systems.
>>
>> ismail
>
>
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r271689 - Fix test for LLVM_LIBDIR_SUFFIX=64 case.

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Fri Jun  3 12:06:52 2016
New Revision: 271689

URL: http://llvm.org/viewvc/llvm-project?rev=271689&view=rev
Log:
Fix test for LLVM_LIBDIR_SUFFIX=64 case.


Modified:
cfe/trunk/test/Driver/android-ndk-standalone.cpp

Modified: cfe/trunk/test/Driver/android-ndk-standalone.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/android-ndk-standalone.cpp?rev=271689&r1=271688&r2=271689&view=diff
==
--- cfe/trunk/test/Driver/android-ndk-standalone.cpp (original)
+++ cfe/trunk/test/Driver/android-ndk-standalone.cpp Fri Jun  3 12:06:52 2016
@@ -17,7 +17,7 @@
 // CHECK-NOT: "-internal-isystem" 
"{{.*}}/include/c++/4.9/arm-linux-androideabi/thumb"
 // CHECK: "-internal-isystem" "{{.*}}/include/c++/4.9/backward"
 // CHECK: "-internal-isystem" "{{.*}}/sysroot/usr/local/include"
-// CHECK: "-internal-isystem" 
"{{.*(/|)}}lib{{(/|)}}clang{{(/|)[^"]+(/|)}}include"
+// CHECK: "-internal-isystem" 
"{{.*(/|)}}lib{{(64)?(/|)}}clang{{(/|)[^"]+(/|)}}include"
 // CHECK: "-internal-externc-isystem" "{{.*}}/sysroot/include"
 // CHECK: "-internal-externc-isystem" "{{.*}}/sysroot/usr/include"
 // CHECK: "{{.*}}ld{{(.exe)?}}" "--sysroot=[[SYSROOT:[^"]+]]"
@@ -53,7 +53,7 @@
 // CHECK-ARMV7-NOT: "-internal-isystem" 
"{{.*}}/include/c++/4.9/arm-linux-androideabi"
 // CHECK-ARMV7: "-internal-isystem" "{{.*}}/include/c++/4.9/backward"
 // CHECK-ARMV7: "-internal-isystem" "{{.*}}/sysroot/usr/local/include"
-// CHECK-ARMV7: "-internal-isystem" 
"{{.*(/|)}}lib{{(/|)}}clang{{(/|)[^"]+(/|)}}include"
+// CHECK-ARMV7: "-internal-isystem" 
"{{.*(/|)}}lib{{(64)?(/|)}}clang{{(/|)[^"]+(/|)}}include"
 // CHECK-ARMV7: "-internal-externc-isystem" "{{.*}}/sysroot/include"
 // CHECK-ARMV7: "-internal-externc-isystem" "{{.*}}/sysroot/usr/include"
 // CHECK-ARMV7: "{{.*}}ld{{(.exe)?}}" "--sysroot=[[SYSROOT:[^"]+]]"
@@ -111,7 +111,7 @@
 // CHECK-THUMB-NOT: "-internal-isystem" 
"{{.*}}/include/c++/4.9/arm-linux-androideabi"
 // CHECK-THUMB: "-internal-isystem" "{{.*}}/include/c++/4.9/backward"
 // CHECK-THUMB: "-internal-isystem" "{{.*}}/sysroot/usr/local/include"
-// CHECK-THUMB: "-internal-isystem" 
"{{.*(/|)}}lib{{(/|)}}clang{{(/|)[^"]+(/|)}}include"
+// CHECK-THUMB: "-internal-isystem" 
"{{.*(/|)}}lib{{(64)?(/|)}}clang{{(/|)[^"]+(/|)}}include"
 // CHECK-THUMB: "-internal-externc-isystem" "{{.*}}/sysroot/include"
 // CHECK-THUMB: "-internal-externc-isystem" "{{.*}}/sysroot/usr/include"
 // CHECK-THUMB: "{{.*}}ld{{(.exe)?}}" "--sysroot=[[SYSROOT:[^"]+]]"
@@ -149,7 +149,7 @@
 // CHECK-ARMV7THUMB-NOT: "-internal-isystem" 
"{{.*}}/include/c++/4.9/arm-linux-androideabi"
 // CHECK-ARMV7THUMB: "-internal-isystem" "{{.*}}/include/c++/4.9/backward"
 // CHECK-ARMV7THUMB: "-internal-isystem" "{{.*}}/sysroot/usr/local/include"
-// CHECK-ARMV7THUMB: "-internal-isystem" 
"{{.*(/|)}}lib{{(/|)}}clang{{(/|)[^"]+(/|)}}include"
+// CHECK-ARMV7THUMB: "-internal-isystem" 
"{{.*(/|)}}lib{{(64)?(/|)}}clang{{(/|)[^"]+(/|)}}include"
 // CHECK-ARMV7THUMB: "-internal-externc-isystem" "{{.*}}/sysroot/include"
 // CHECK-ARMV7THUMB: "-internal-externc-isystem" "{{.*}}/sysroot/usr/include"
 // CHECK-ARMV7THUMB: "{{.*}}ld{{(.exe)?}}" "--sysroot=[[SYSROOT:[^"]+]]"


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r271427 - [driver][arm] add armv7 and thumb include and lib paths

[Ismail Donmez via cfe-commits]

Fixed in r271689.

On Fri, Jun 3, 2016 at 9:55 AM, Ismail Donmez  wrote:
> This is due to lib vs lib64. Will fix. Sorry for the wrong diagnosis :)
>
> On Thu, Jun 2, 2016 at 8:24 PM, Chih-hung Hsieh  wrote:
>> Ismail,
>> I saw that android-ndk-standalone.cpp has already -stdlib=libstdc++.
>> Could you tell me or just go ahead and make the necessary change?
>> Thanks.
>>
>>
>> On Thu, Jun 2, 2016 at 9:52 AM, Ismail Donmez  wrote:
>>>
>>> Hi,
>>>
>>> On Wed, Jun 1, 2016 at 11:48 PM, Chih-Hung Hsieh via cfe-commits
>>>  wrote:
>>> > Author: chh
>>> > Date: Wed Jun  1 15:48:46 2016
>>> > New Revision: 271427
>>> >
>>> > URL: http://llvm.org/viewvc/llvm-project?rev=271427&view=rev
>>> > Log:
>>> > [driver][arm] add armv7 and thumb include and lib paths
>>> >
>>> > Add a new test android-ndk-standalone.cpp
>>> > with new Android NDK release tree structure.
>>> > Detect armv7 sub architecture and thumb mode,
>>> > to add system include and link search paths.
>>> >
>>> > Differential Revision: http://reviews.llvm.org/D20600
>>> >
>>>
>>> android-ndk-standalone.cpp needs -stdlib=libstdc++ to fix test on
>>> libc++ default systems.
>>>
>>> ismail
>>
>>
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r271801 - Add PIE magic for NetBSD. Add tests for the correct flags for

[Ismail Donmez via cfe-commits]

Hi,

On Sat, Jun 4, 2016 at 11:03 PM, Joerg Sonnenberger via cfe-commits
 wrote:
> Author: joerg
> Date: Sat Jun  4 15:03:26 2016
> New Revision: 271801
>
> URL: http://llvm.org/viewvc/llvm-project?rev=271801&view=rev
> Log:
> Add PIE magic for NetBSD. Add tests for the correct flags for
> non-shared, PIE and shared output mode.
>
> Modified:
> cfe/trunk/lib/Driver/Tools.cpp
> cfe/trunk/test/Driver/netbsd.c

This seems to fail for me on Windows:
$ "C:/cygwin64/home/ismail/src/llvm/dist/./bin/clang.EXE"
"-no-canonical-prefixes" "-target" "x86_64--netbsd"
"--sysroot=C:\cygwin64\home\ismail\src\llvm\tools\clang\test\Driver/Inputs/basic_netbsd_tree"
"C:\cygwin64\home\ismail\src\llvm\tools\clang\test\Driver\netbsd.c"
"-###"
$ "C:/cygwin64/home/ismail/src/llvm/dist/./bin\FileCheck.EXE"
"-check-prefix=STATIC"
"C:\cygwin64\home\ismail\src\llvm\tools\clang\test\Driver\netbsd.c"
# command stderr:
C:\cygwin64\home\ismail\src\llvm\tools\clang\test\Driver\netbsd.c:121:12:
error: expected string not found in input
// STATIC: "-dynamic-linker" "/libexec/ld.elf_so"
   ^
:7:1: note: scanning from here

^

error: command failed with exit status: 1

--



Here is the output passed to FileCheck:

win7-vbox ~ > "C:/cygwin64/home/ismail/src/llvm/dist/./bin/clang.EXE"
"-no-canonical-prefixes" "-target" "x86_64--netbsd"
"--sysroot=C:\cygwin64\home\ismail\src\llvm\tools\clang\test\Driver/Inputs/basic_netbsd_tree"
"C:\cygwin64\home\ismail\src\llvm\tools\clang\test\Driver\netbsd.c"
"-###"
clang version 3.9.0 (http://llvm.org/git/clang
4076bbb87bd1f3b606192c84d4de201e9b9217fb) (http://llvm.org/git/llvm
6dbfac925fe759193c6a16089e2be36f586729d6)
Target: x86_64--netbsd
Thread model: posix
InstalledDir: C:\cygwin64\home\ismail\src\llvm\dist\bin
 "C:\\cygwin64\\home\\ismail\\src\\llvm\\dist\\bin\\clang.EXE" "-cc1"
"-triple" "x86_64--netbsd" "-emit-obj" "-mrelax-all" "-disable-free"
"-disable-llvm-verifier" "-discard-value-names" "-main-file-name"
"netbsd.c" "-mrelocation-model" "static" "-mthread-model" "posix"
"-mdisable-fp-elim" "-masm-verbose" "-mconstructor-aliases"
"-munwind-tables" "-target-cpu" "x86-64" "-dwarf-column-info"
"-debugger-tuning=gdb" "-resource-dir"
"C:\\cygwin64\\home\\ismail\\src\\llvm\\dist\\bin\\..\\lib\\clang\\3.9.0"
"-isysroot" 
"C:\\cygwin64\\home\\ismail\\src\\llvm\\tools\\clang\\test\\Driver/Inputs/basic_netbsd_tree"
"-fdebug-compilation-dir" "C:\\cygwin64\\home\\ismail" "-ferror-limit"
"19" "-fmessage-length" "0" "-fobjc-runtime=gnustep"
"-fdiagnostics-show-option" "-o" "C:\\cygwin64\\tmp\\netbsd-7165c6.o"
"-x" "c" 
"C:\\cygwin64\\home\\ismail\\src\\llvm\\tools\\clang\\test\\Driver\\netbsd.c"
 "C:\\cygwin64\\bin\\ld.exe"
"--sysroot=C:\\cygwin64\\home\\ismail\\src\\llvm\\tools\\clang\\test\\Driver/Inputs/basic_netbsd_tree"
"--eh-frame-hdr" "-dynamic-linker" "/libexec/ld.elf_so" "-o" "a.out"
"C:\\cygwin64\\home\\ismail\\src\\llvm\\tools\\clang\\test\\Driver/Inputs/basic_netbsd_tree/usr/lib\\crt0.o"
"C:\\cygwin64\\home\\ismail\\src\\llvm\\tools\\clang\\test\\Driver/Inputs/basic_netbsd_tree/usr/lib\\crti.o"
"C:\\cygwin64\\home\\ismail\\src\\llvm\\tools\\clang\\test\\Driver/Inputs/basic_netbsd_tree/usr/lib\\crtbegin.o"
"C:\\cygwin64\\tmp\\netbsd-7165c6.o" "-lc"
"C:\\cygwin64\\home\\ismail\\src\\llvm\\tools\\clang\\test\\Driver/Inputs/basic_netbsd_tree/usr/lib\\crtend.o"
"C:\\cygwin64\\home\\ismail\\src\\llvm\\tools\\clang\\test\\Driver/Inputs/basic_netbsd_tree/usr/lib\\crtn.o"

Let me know if you can stop any error.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

[Ismail Donmez via cfe-commits]

Hi,

On Mon, Jun 20, 2016 at 10:26 PM, Yaxun Liu via cfe-commits
 wrote:
> Author: yaxunl
> Date: Mon Jun 20 14:26:00 2016
> New Revision: 273191
>
> URL: http://llvm.org/viewvc/llvm-project?rev=273191&view=rev
> Log:
> [OpenCL] Include opencl-c.h by default as a clang module
>
> Include opencl-c.h by default as a module to utilize the automatic AST 
> caching mechanism of clang modules.
>
> Add an option -finclude-default-header to enable default header for OpenCL, 
> which is off by default.
>
> Differential Revision: http://reviews.llvm.org/D20444
>
> Modified:
> cfe/trunk/include/clang/Basic/LangOptions.def
> cfe/trunk/include/clang/Driver/CC1Options.td
> cfe/trunk/include/clang/Frontend/CompilerInvocation.h
> cfe/trunk/lib/Frontend/CompilerInvocation.cpp
> cfe/trunk/lib/Headers/module.modulemap
> cfe/trunk/test/Headers/opencl-c-header.cl

chmod lines doesn't seem to work on Cygwin:

$ "chmod" "u-w"
"C:\cygwin64\home\ismail\src\llvm\dist\tools\clang\test\Headers\Output\opencl-c-header.cl.tmp/*"
# command stderr:
r.cl.tmp/*: invalid mode: 'mp/*'
Try 'r.cl.tmp/* --help' for more information.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

[Ismail Donmez via cfe-commits]

On Wed, Jun 22, 2016 at 5:45 PM, Liu, Yaxun (Sam)  wrote:
> $ "chmod" "u-w"
> "C:\cygwin64\home\ismail\src\llvm\dist\tools\clang\test\Headers\Output\opencl-c-header.cl.tmp/*"
> # command stderr:
> r.cl.tmp/*: invalid mode: 'mp/*'
>
> the error msg is strange. On my Cygwin I got different error:
> chmod: cannot access 
> 'C:\cygwin64\home\ismail\src\llvm\dist\tools\clang\test\Headers\Output\opencl-c-header.cl.tmp/*':
>  No such file or directory
>
> This is because * is not expanded when quoted.

Actually I added the quotes to test if it fixes the issue. Since you
have cygwin can you reproduce the error?

ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

[Ismail Donmez via cfe-commits]

Hi,

On Wed, Jun 22, 2016 at 7:55 PM, Liu, Yaxun (Sam)  wrote:
> The cmake of Cygwin itself does not support llvm. Which cmake did you use on 
> Cygwin? Thanks.

I use native windows cmake and this is the only test that ever failed
with this setup. chmod is from native cygwin which seems to be the
problem.

ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

[Ismail Donmez via cfe-commits]

Hi,

I won't have access to the system for ~2 weeks but the change looks
fine. Please commit if it passes the test on Linux with that.

Regards,
ismail


On Thu, Jun 23, 2016 at 4:39 PM, Liu, Yaxun (Sam)  wrote:
> I have a patch which may workaround this issue, however I could not test it 
> in Cygwin since I got issues build latest cmake in Cygwin and the downloaded 
> cmake does not work.
>
> Could you please try it?
>
> Thanks.
>
> Sam
>
> -Original Message-
> From: Ismail Donmez [mailto:ism...@i10z.com]
> Sent: Wednesday, June 22, 2016 1:52 PM
> To: Liu, Yaxun (Sam) 
> Cc: cfe-commits 
> Subject: Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang 
> module
>
> Hi,
>
> On Wed, Jun 22, 2016 at 7:55 PM, Liu, Yaxun (Sam)  wrote:
>> The cmake of Cygwin itself does not support llvm. Which cmake did you use on 
>> Cygwin? Thanks.
>
> I use native windows cmake and this is the only test that ever failed with 
> this setup. chmod is from native cygwin which seems to be the problem.
>
> ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r257260 - [vfs] Normalize working directory if requested.

[Ismail Donmez via cfe-commits]

Hi,

> +  NormalizedFS.setCurrentWorkingDirectory("/b/c");
> +  NormalizedFS.setCurrentWorkingDirectory(".");
> +  ASSERT_EQ("/b/c", NormalizedFS.getCurrentWorkingDirectory().get());
> +  NormalizedFS.setCurrentWorkingDirectory("..");
> +  ASSERT_EQ("/b", NormalizedFS.getCurrentWorkingDirectory().get());
>  }

This test fails on Windows (MSVC 2015 x64):

 TEST 'Clang-Unit ::
Basic/BasicTests.exe/InMemoryFileSystemTest.WorkingDirectory' FAILED


Note: Google Test filter = InMemoryFileSystemTest.WorkingDirectory
[==] Running 1 test from 1 test case.
[--] Global test environment set-up.
[--] 1 test from InMemoryFileSystemTest
[ RUN  ] InMemoryFileSystemTest.WorkingDirectory
..\tools\clang\unittests\Basic\VirtualFileSystemTest.cpp(663): error:
Value of: NormalizedFS.getCurrentWorkingDirectory().get(
)
  Actual: "/b\\c"
Expected: "/b/c"
[  FAILED  ] InMemoryFileSystemTest.WorkingDirectory (0 ms)
[--] 1 test from InMemoryFileSystemTest (0 ms total)

[--] Global test environment tear-down
[==] 1 test from 1 test case ran. (0 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] InMemoryFileSystemTest.WorkingDirectory

 1 FAILED TEST


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r257468 - Revert r253898 and r253899, this breaks mingw compilation on openSUSE.

[Ismail Donmez via cfe-commits]

Author: ismail
Date: Tue Jan 12 04:41:20 2016
New Revision: 257468

URL: http://llvm.org/viewvc/llvm-project?rev=257468&view=rev
Log:
Revert r253898 and r253899, this breaks mingw compilation on openSUSE.
Will re-apply after llvm 3.8 is branched. 

Original commit message:

Driver: fallback to the location of clang if no sysroot,

hard coding /usr makes little sense for mingw-w64.
If we have portable toolchains having /usr breaks that.
If the clang we use is in /usr/bin or /usr/sbin etc this will
still detect as though it was hard coded to /usr

This makes the most sense going forward for mingw-w64 toolchains
on both linux and mac


Modified:
cfe/trunk/lib/Driver/MinGWToolChain.cpp

Modified: cfe/trunk/lib/Driver/MinGWToolChain.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/MinGWToolChain.cpp?rev=257468&r1=257467&r2=257468&view=diff
==
--- cfe/trunk/lib/Driver/MinGWToolChain.cpp (original)
+++ cfe/trunk/lib/Driver/MinGWToolChain.cpp Tue Jan 12 04:41:20 2016
@@ -66,17 +66,23 @@ MinGW::MinGW(const Driver &D, const llvm
 : ToolChain(D, Triple, Args) {
   getProgramPaths().push_back(getDriver().getInstalledDir());
 
-  // On Windows if there is no sysroot we search for gcc on the PATH.
-  if (getDriver().SysRoot.size())
-  Base = getDriver().SysRoot;
+// In Windows there aren't any standard install locations, we search
+// for gcc on the PATH. In Linux the base is always /usr.
 #ifdef LLVM_ON_WIN32
+  if (getDriver().SysRoot.size())
+Base = getDriver().SysRoot;
   else if (llvm::ErrorOr GPPName =
llvm::sys::findProgramByName("gcc"))
 Base = llvm::sys::path::parent_path(
 llvm::sys::path::parent_path(GPPName.get()));
-#endif
-  if (!Base.size())
+  else
 Base = llvm::sys::path::parent_path(getDriver().getInstalledDir());
+#else
+  if (getDriver().SysRoot.size())
+Base = getDriver().SysRoot;
+  else
+Base = "/usr";
+#endif
 
   Base += llvm::sys::path::get_separator();
   findGccLibDir();


___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r257533 - D9600: Add scan-build python implementation

[Ismail Donmez via cfe-commits]

Hi,

On Wed, Jan 13, 2016 at 12:38 AM, Laszlo Nagy via cfe-commits
 wrote:
> Author: rizsotto
> Date: Tue Jan 12 16:38:41 2016
> New Revision: 257533
>
> URL: http://llvm.org/viewvc/llvm-project?rev=257533&view=rev
> Log:
> D9600: Add scan-build python implementation

This doesn't seem to be installed by default, is that intended?

Regards,
ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r257533 - D9600: Add scan-build python implementation

[Ismail Donmez via cfe-commits]

Thanks for the clarification.

On Thursday, January 14, 2016, Laszlo Nagy 
wrote:

> yes, it is just an initial drop. improvements are planed to make a full
> replacement of the one written in Perl.
>
> On Thu, Jan 14, 2016 at 12:14 AM, Ismail Donmez  > wrote:
>
>> Hi,
>>
>> On Wed, Jan 13, 2016 at 12:38 AM, Laszlo Nagy via cfe-commits
>> > > wrote:
>> > Author: rizsotto
>> > Date: Tue Jan 12 16:38:41 2016
>> > New Revision: 257533
>> >
>> > URL: http://llvm.org/viewvc/llvm-project?rev=257533&view=rev
>> > Log:
>> > D9600: Add scan-build python implementation
>>
>> This doesn't seem to be installed by default, is that intended?
>>
>> Regards,
>> ismail
>>
>
>
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r257982 - [libclang] Add missing CINDEX_LINKAGE from some new APIs in Index.h.

[Ismail Donmez via cfe-commits]

On Sat, Jan 16, 2016 at 5:01 AM, Argyrios Kyrtzidis via cfe-commits
 wrote:
> Author: akirtzidis
> Date: Fri Jan 15 21:01:20 2016
> New Revision: 257982
>
> URL: http://llvm.org/viewvc/llvm-project?rev=257982&view=rev
> Log:
> [libclang] Add missing CINDEX_LINKAGE from some new APIs in Index.h.
>
> Modified:
> cfe/trunk/include/clang-c/Index.h

Should this be in 3.8 branch too?

ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r253898 - Driver: fallback to the location of clang if no sysroot,

[Ismail Donmez via cfe-commits]

Hi,

On Mon, Nov 23, 2015 at 8:59 PM, Martell Malone via cfe-commits
 wrote:
> Author: martell
> Date: Mon Nov 23 12:59:48 2015
> New Revision: 253898
>
> URL: http://llvm.org/viewvc/llvm-project?rev=253898&view=rev
> Log:
> Driver: fallback to the location of clang if no sysroot,
>
> hard coding /usr makes little sense for mingw-w64.
> If we have portable toolchains having /usr breaks that.
> If the clang we use is in /usr/bin or /usr/sbin etc this will
> still detect as though it was hard coded to /usr
>
> This makes the most sense going forward for mingw-w64 toolchains
> on both linux and mac

This breaks mingw support on openSUSE :

λ cat hello.c
#include 

int main()
{
return 0;
}

λ clang -v -target x86_64-w64-mingw32 hello.c
clang version 3.8.0 (trunk 253903)
Target: x86_64-w64-windows-gnu
Thread model: posix
InstalledDir: /opt/clang/bin
 "/opt/clang/bin/clang-3.8" -cc1 -triple x86_64-w64-windows-gnu
-emit-obj -mrelax-all -disable-free -main-file-name hello.c
-mrelocation-model pic -pic-level 2 -mthread-model posix -fmath-errno
-masm-verbose -mconstructor-aliases -munwind-tables -target-cpu x86-64
-momit-leaf-frame-pointer -v -dwarf-column-info -resource-dir
/opt/clang/bin/../lib64/clang/3.8.0 -internal-isystem
/opt/clang/bin/../lib64/clang/3.8.0/include -internal-isystem include
-internal-isystem /opt/clang/x86_64-w64-mingw32/sys-root/mingw/include
-internal-isystem include-fixed -internal-isystem
/opt/clang/x86_64-w64-mingw32/include -internal-isystem
/opt/clang/include -fdebug-compilation-dir /home/ismail -ferror-limit
19 -fmessage-length 127 -fno-use-cxa-atexit -fobjc-runtime=gcc
-fdiagnostics-show-option -fcolor-diagnostics -o /tmp/hello-f129aa.o
-x c hello.c
clang -cc1 version 3.8.0 based upon LLVM 3.8.0svn default target
x86_64-suse-linux
ignoring nonexistent directory "include"
ignoring nonexistent directory
"/opt/clang/x86_64-w64-mingw32/sys-root/mingw/include"
ignoring nonexistent directory "include-fixed"
ignoring nonexistent directory "/opt/clang/x86_64-w64-mingw32/include"
#include "..." search starts here:
#include <...> search starts here:
 /opt/clang/bin/../lib64/clang/3.8.0/include
 /opt/clang/include
End of search list.
hello.c:1:10: fatal error: 'stdlib.h' file not found
#include 
 ^
1 error generated.

x86_64-w64-mingw32-gcc is in $PATH, and this used to work fine before
this commit.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r253898 - Driver: fallback to the location of clang if no sysroot,

[Ismail Donmez via cfe-commits]

On Tue, Nov 24, 2015 at 12:43 PM, Martell Malone
 wrote:
>> This breaks mingw support on openSUSE :
>
> My first question is why on SUSE is clang installed in /opt while mingw-w64
> in /usr?

Well this is a custom clang it can be anywhere. Official one is in /usr.

>> x86_64-w64-mingw32-gcc is in $PATH, and this used to work fine before
>> this commit.
>
> It doesn't look for gcc on linux that is a windows host only thing.
> It didn't do that before this commit also.
> SUSE was just lucky because we hard coded /usr as the base path.

This is not a SUSE only thing, afaik Fedora has the same setup.

> I don't like the idea of hard coding for just a single distro so I think
> We could optionally do some search for "x86_64-w64-mingw32-gcc" on non
> windows hosts
> Just like we do for "gcc" on windows hosts.
> This should fix SUSE while maintaining the new more reasonable search
> pattern.

Why not hardcode /usr for Linux hosts?

Thanks,
ismail
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

[Ismail Donmez via cfe-commits]

Hi,

On Tue, Dec 1, 2015 at 7:45 PM, Reid Kleckner via cfe-commits
 wrote:
> rnk added a comment.
>
> In http://reviews.llvm.org/D14180#299183, @martell wrote:
>
>> I tried testing __has_builtin(_InterlockedCompareExchangeAdd) as it seem to 
>> not exist even when -fms-extensions is passed
>>  Is this a bug or it is intended ?
>
>
> Neither Clang nor MSDN have anything by that name. Do you mean 
> _InterlockedExchangeAdd? __has_builtin should find that.

MSDN knows about it though:
https://msdn.microsoft.com/en-us/library/windows/desktop/ttk2z1ws(v=vs.85).aspx
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D14180: enable -fms-extensions by default on the mingw-w64 target

[Ismail Donmez via cfe-commits]

On Tue, Dec 1, 2015 at 8:06 PM, Ismail Donmez  wrote:
> Hi,
>
> On Tue, Dec 1, 2015 at 7:45 PM, Reid Kleckner via cfe-commits
>  wrote:
>> rnk added a comment.
>>
>> In http://reviews.llvm.org/D14180#299183, @martell wrote:
>>
>>> I tried testing __has_builtin(_InterlockedCompareExchangeAdd) as it seem to 
>>> not exist even when -fms-extensions is passed
>>>  Is this a bug or it is intended ?
>>
>>
>> Neither Clang nor MSDN have anything by that name. Do you mean 
>> _InterlockedExchangeAdd? __has_builtin should find that.
>
> MSDN knows about it though:
> https://msdn.microsoft.com/en-us/library/windows/desktop/ttk2z1ws(v=vs.85).aspx

Never mind, misread the function name.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits