Re: [PATCH] D18035: [GCC] PR23529 Mangler part of attrbute abi_tag support

2016-06-02 Thread Hal Finkel via cfe-commits
hfinkel added a subscriber: hfinkel. hfinkel added a comment. FYI: There is now a specification, see: http://sourcerytools.com/pipermail/cxx-abi-dev/2016-June/002919.html http://reviews.llvm.org/D18035 ___ cfe-commits mailing list cfe-commits@lists

cmake patch for building latest clang vs latest llvm

2016-06-02 Thread Rudolf Kastl via cfe-commits
This patch adjusts the path for the cmake directory to work properly on systems with libsuffix=64. Not sure if it is the preferred way of fixing that but it fixed the build for me. --- CMakeLists.txt.old 2016-06-03 00:19:08.676041254 +0200 +++ CMakeLists.txt 2016-06-03 00:19:48.778920468 +0200 @@ -

Re: Don't pass --build-id by default.

2016-06-02 Thread Hal Finkel via cfe-commits
- Original Message - > From: "Rafael EspĂ­ndola" > To: "Hal Finkel" > Cc: "cfe-commits cfe" > Sent: Thursday, June 2, 2016 7:06:26 PM > Subject: Re: Don't pass --build-id by default. > > > This is going to break a lot of my local rpm packaging scripts, and > > I suspect the same is true

Re: [PATCH] D20909: [clang-tidy] Ignore function context in misc-unused-using-decls.

2016-06-02 Thread Alexander Kornienko via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG Comment at: clang-tidy/misc/UnusedUsingDeclsCheck.cpp:109 @@ -100,1 +108,3 @@ + // giving an incorrect message, we mark all of them as used. for (auto &Context : Contex

Re: [PATCH] D20929: [libcxx] Fix join.pass.cpp segfault after r271475

2016-06-02 Thread Eric Fiselier via cfe-commits
EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land. Thanks for the patch. http://reviews.llvm.org/D20929 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailma

Re: [PATCH] D20942: [LockFileManager] Improve error output by adding error messages

2016-06-02 Thread Ben Langmuir via cfe-commits
benlangmuir accepted this revision. benlangmuir added a comment. This revision is now accepted and ready to land. LGTM. Some minor suggestions. Comment at: include/llvm/Support/LockFileManager.h:87 @@ +86,3 @@ + + /// \brief Get error message + std::string getErrorMessage() c

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

2016-06-02 Thread 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 Th

<    1   2