Thanks Alex...
On Fri, Jun 9, 2017 at 1:02 AM, Alexander Kornienko via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Fixes committed in r305024 and r305057.
>
> On Fri, Jun 9, 2017 at 12:07 AM, Alexander Kornienko
> wrote:
>
>> It looks like the buildbots have exceptions turned off by defau
On Sat, Jun 10, 2017 at 8:55 PM John McCall via Phabricator <
revi...@reviews.llvm.org> wrote:
> rjmccall accepted this revision.
> rjmccall added a comment.
> This revision is now accepted and ready to land.
>
> Thanks, looks great.
Great, thanks.
>
>
> If you're going to be submitting multipl
Author: dhinton
Date: Thu Dec 14 14:12:46 2017
New Revision: 320751
URL: http://llvm.org/viewvc/llvm-project?rev=320751&view=rev
Log:
[debuginfo] Remove temporary FIXME.
Summary:
Now that r320495, "[debuginfo-tests] Support moving
debuginfo-tests to llvm/projects," has landed, remove temporary FI
Author: dhinton
Date: Sun Dec 24 21:33:42 2017
New Revision: 321435
URL: http://llvm.org/viewvc/llvm-project?rev=321435&view=rev
Log:
[libcxx] Suppress unused warning on apple.
Summary:
This warning is already suppressed on non-apple platforms, so
this change just suppresses it on apple as well.
Hi:
This change breaks in a local debug build, e.g.,:
/Users/dhinton/projects/llvm_project/libunwind/src/DwarfParser.hpp:559:28:
error: no member named 'numeric_limits' in namespace 'std'
assert(length < std::numeric_limits::max() && "pointer
overflow");
~^
thanks
On Mon, Dec 25, 2017 at 11:09 AM, whitequark
wrote:
> On 2017-12-25 19:04, Don Hinton wrote:
>
>> Hi:
>>
>> This change breaks in a local debug build, e.g.,:
>>
>> /Users/dhinton/projects/llvm_project/libunwind/src/DwarfPars
>> er.hpp:559:28:
>> error: no member named 'numeric_limits' in namespac
Here's the patch I applied locally.
hth...
don
diff --git a/src/DwarfParser.hpp b/src/DwarfParser.hpp
index 518101e..ac4f1c4 100644
--- a/src/DwarfParser.hpp
+++ b/src/DwarfParser.hpp
@@ -540,7 +540,7 @@ bool CFI_Parser::parseInstructions(A &addressSpace,
pint_t instructions,
results->cfa
While beauty is in the eye of the beholder, I'm not sure introducing a new
template function in a header that's only used in that header is a good
idea. Every file that includes DwarfParser.hpp is going to get that
template function -- and someone may try to use it someday.
However, if you did wa
On Mon, Dec 25, 2017 at 12:43 PM, whitequark
wrote:
> On 2017-12-25 20:32, Don Hinton wrote:
>
>> While beauty is in the eye of the beholder, I'm not sure introducing a
>> new template function in a header that's only used in that header is a
>> good idea. Every file that includes DwarfParser.hp
On Mon, Dec 25, 2017 at 1:10 PM, whitequark
wrote:
> On 2017-12-25 20:54, Don Hinton wrote:
>
>> On Mon, Dec 25, 2017 at 12:43 PM, whitequark
>> wrote:
>>
>>> On 2017-12-25 20:32, Don Hinton wrote:
>>>
It should also assert for non-integral types.
>>>
>>> True, but I do not know enough
Author: dhinton
Date: Tue Jan 9 17:00:28 2018
New Revision: 322147
URL: http://llvm.org/viewvc/llvm-project?rev=322147&view=rev
Log:
[cmake] Delete redundant install command for clang-refactor.
Summary: Install targets for clang tools are controlled by
CLANG_BUILD_TOOLS, and when OFF, cmake issu
It looks like this change introduced a small bug; Specifically, the
following cast test:
- if (auto PT = dyn_cast(DestTy)) {
...
+ // If we're producing a pointer, this is easy.
+ if (auto destPtrTy = cast(destTy)) {
Since the cast can fail, shouldn't you prefer dyn_cast<>(), which can
re
Hi John:
I found this investigating the cast assert noted here:
http://lists.llvm.org/pipermail/cfe-dev/2019-March/061685.html
I subsequently did quick grep and found a number of cases in clang+llvm
(didn't find any in other projects) . I'm happy to fix these in mass,
i.e., s/cast/dyn_cast/, but
Author: dhinton
Date: Mon Mar 18 23:14:14 2019
New Revision: 356441
URL: http://llvm.org/viewvc/llvm-project?rev=356441&view=rev
Log:
Refactor cast<>'s in if conditionals, which can only assert on failure.
Summary:
This patch refactors several instances of cast<> used in if
conditionals. Since c
Author: dhinton
Date: Sat Apr 13 09:55:28 2019
New Revision: 358337
URL: http://llvm.org/viewvc/llvm-project?rev=358337&view=rev
Log:
[CommandLineParser] Add DefaultOption flag
Summary: Add DefaultOption flag to CommandLineParser which provides a
default option or alias, but allows users to overr
Author: dhinton
Date: Mon Apr 15 10:18:10 2019
New Revision: 358428
URL: http://llvm.org/viewvc/llvm-project?rev=358428&view=rev
Log:
[CommandLineParser] Add DefaultOption flag
Summary: Add DefaultOption flag to CommandLineParser which provides a
default option or alias, but allows users to overr
Author: dhinton
Date: Wed Apr 24 14:25:57 2019
New Revision: 359142
URL: http://llvm.org/viewvc/llvm-project?rev=359142&view=rev
Log:
[clang-tidy] Add new checker: llvm-prefer-isa-or-dyn-cast-in-conditionals
Summary:
Looks at conditionals and finds cases of ``cast<>``, which will
assert rather th
Author: dhinton
Date: Fri May 3 10:47:29 2019
New Revision: 359909
URL: http://llvm.org/viewvc/llvm-project?rev=359909&view=rev
Log:
[CommandLine] Change help output to prefix long options with `--` instead of
`-`. NFC . Part 3 of 5
Summary:
By default, `parseCommandLineOptions()` will accept e
Author: dhinton
Date: Sun Jun 2 08:53:43 2019
New Revision: 362328
URL: http://llvm.org/viewvc/llvm-project?rev=362328&view=rev
Log:
[test] Fix plugin tests
Recommit of r361790 that was temporarily reverted in r361793 due to bot
breakage.
Summary:
The following changes were required to fix the
Author: dhinton
Date: Sun Jun 2 10:56:26 2019
New Revision: 362334
URL: http://llvm.org/viewvc/llvm-project?rev=362334&view=rev
Log:
Fix bug in r362328 -- append to dependency list instead of overwrite it.
Modified:
cfe/trunk/test/CMakeLists.txt
Modified: cfe/trunk/test/CMakeLists.txt
URL:
Author: dhinton
Date: Tue Jun 4 15:07:40 2019
New Revision: 362555
URL: http://llvm.org/viewvc/llvm-project?rev=362555&view=rev
Log:
[Analysis] Only build Analysis plugins when CLANG_ENABLE_STATIC_ANALYZER is
enabled.
Fixes bug introduced in r362328.
Thanks to Nathan Chancellor for reporting t
Author: dhinton
Date: Thu Jun 13 12:08:49 2019
New Revision: 363298
URL: http://llvm.org/viewvc/llvm-project?rev=363298&view=rev
Log:
[lit] Disable test on darwin when building shared libs.
Summary:
This test fails to link shared libraries because tries to run
a copied version of clang-check to s
Author: dhinton
Date: Sat Jun 15 13:09:54 2019
New Revision: 363502
URL: http://llvm.org/viewvc/llvm-project?rev=363502&view=rev
Log:
Recommit r363298 "[lit] Disable test on darwin when building shared libs."
Was reverted in r363379 due to build breakage.
Thanks to Nico Weber for reverting the o
Author: dhinton
Date: Sat Jun 15 18:09:41 2019
New Revision: 363504
URL: http://llvm.org/viewvc/llvm-project?rev=363504&view=rev
Log:
Fix gcc-05.4 bot failures caused by in r363481 "[clangd] Index API and
implementations for relations"
Use std::make_tuple instead of initializer list to make gcc-
Author: dhinton
Date: Sun Jun 16 10:57:37 2019
New Revision: 363518
URL: http://llvm.org/viewvc/llvm-project?rev=363518&view=rev
Log:
[docs] Fix a few problems with clang-tool docs to get the bots green again.
Modified:
clang-tools-extra/trunk/docs/ReleaseNotes.rst
clang-tools-extra/trunk
Author: dhinton
Date: Sun Jun 16 11:41:31 2019
New Revision: 363520
URL: http://llvm.org/viewvc/llvm-project?rev=363520&view=rev
Log:
[docs] Fix another bot warning by adding a blank line to separate the
`option::` command from the text below.
Modified:
clang-tools-extra/trunk/docs/clang-ti
Author: dhinton
Date: Sun Jun 16 12:15:04 2019
New Revision: 363521
URL: http://llvm.org/viewvc/llvm-project?rev=363521&view=rev
Log:
[docs] Fix another bot error by setting highlight language of objc code-block
to objc instead of c++.
Modified:
cfe/trunk/docs/ReleaseNotes.rst
Modified: cfe
Just saw this. I'll take a look, and thanks for pointing it out...
On Fri, Aug 2, 2019 at 6:24 PM Artem Dergachev via Phabricator <
revi...@reviews.llvm.org> wrote:
> NoQ added a comment.
>
> Ugh, there seems to be one more forgotten buildbot with plugins problems:
> http://green.lab.llvm.org/gr
Author: dhinton
Date: Wed Jul 10 10:57:05 2019
New Revision: 365675
URL: http://llvm.org/viewvc/llvm-project?rev=365675&view=rev
Log:
Recommit "[CommandLine] Remove OptionCategory and SubCommand caches from the
Option class."
Previously reverted in 364141 due to buildbot breakage, and fixed here
Author: dhinton
Date: Wed Jul 10 10:57:05 2019
New Revision: 365675
URL: http://llvm.org/viewvc/llvm-project?rev=365675&view=rev
Log:
Recommit "[CommandLine] Remove OptionCategory and SubCommand caches from the
Option class."
Previously reverted in 364141 due to buildbot breakage, and fixed here
Author: dhinton
Date: Fri May 10 11:27:09 2019
New Revision: 360450
URL: http://llvm.org/viewvc/llvm-project?rev=360450&view=rev
Log:
[clang-tidy] Change the namespace for llvm checkers from 'llvm' to 'llvm_check'
Summary:
Change the namespace for llvm checkers from 'llvm' to
'llvm_check', and mo
Author: dhinton
Date: Wed May 15 09:58:58 2019
New Revision: 360785
URL: http://llvm.org/viewvc/llvm-project?rev=360785&view=rev
Log:
[clang-tidy] modernize-loop-convert: impl const cast iter
Summary:
modernize-loop-convert was not detecting implicit casts to
const_iterator as convertible to rang
Just wanted to note that this patch was contributed by Torbjörn Klatt, and
I failed to add the following line to the commit message:
Patch by Torbjörn Klatt!
Sorry about that Torbjörn.
On Wed, May 15, 2019 at 9:56 AM Don Hinton via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
&g
Author: dhinton
Date: Wed May 15 10:36:54 2019
New Revision: 360787
URL: http://llvm.org/viewvc/llvm-project?rev=360787&view=rev
Log:
Revert [clang-tidy] modernize-loop-convert: impl const cast iter
This reverts r360785 (git commit 42d28be802fe5beab18bc1a27f89894c0a290d44)
Modified:
clang-to
Author: dhinton
Date: Wed May 15 10:47:51 2019
New Revision: 360788
URL: http://llvm.org/viewvc/llvm-project?rev=360788&view=rev
Log:
[clang-tidy] Recommit r360785 "modernize-loop-convert: impl const cast iter"
with correct attribution
Summary:
modernize-loop-convert was not detecting implicit c
Author: dhinton
Date: Tue May 21 17:56:42 2019
New Revision: 361343
URL: http://llvm.org/viewvc/llvm-project?rev=361343&view=rev
Log:
[Docs] Increase Doxygen cache size
Summary:
When building Doxygen docs for llvm and clang, it helpfully prints a warning at
the end noting that the `LOOKUP_CACHE_S
Author: dhinton
Date: Tue May 21 18:01:11 2019
New Revision: 361344
URL: http://llvm.org/viewvc/llvm-project?rev=361344&view=rev
Log:
[clang-tidy] remove default header-filter for run-clang-tidy
Summary:
run-clang-tidy.py was enforcing '-header-filter' parameter with an
unfortunate default value
Author: dhinton
Date: Fri May 24 12:21:21 2019
New Revision: 361663
URL: http://llvm.org/viewvc/llvm-project?rev=361663&view=rev
Log:
[cmake] Remove old unused version of FindZ3.cmake from clang [NFC]
Summary: This file was moved to llvm in D54978, r356929, but the old
file was never removed.
Re
I've submitted a patch to fix this, https://reviews.llvm.org/D62445.
Without it, non of the bots can build/test any plugins.
On Fri, May 24, 2019 at 5:47 PM Akira Hatanaka via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> I reverted the patch in r361685 to make the bot green again.
>
> > On
Author: dhinton
Date: Mon May 27 23:26:58 2019
New Revision: 361790
URL: http://llvm.org/viewvc/llvm-project?rev=361790&view=rev
Log:
[test] Fix plugin tests
Summary:
The following changes were required to fix these tests:
1) Change LLVM_ENABLE_PLUGINS to an option and move it to
llvm/CMakeLi
Author: dhinton
Date: Mon May 27 23:38:16 2019
New Revision: 361793
URL: http://llvm.org/viewvc/llvm-project?rev=361793&view=rev
Log:
Revert [test] Fix plugin tests
This reverts r361790 (git commit fe5eaab2b5b4523886bd63aebcfea8cfce586fa1)
It's causing buildbot breakage, so reverting while I inv
Author: dhinton
Date: Tue Dec 12 08:48:35 2017
New Revision: 320494
URL: http://llvm.org/viewvc/llvm-project?rev=320494&view=rev
Log:
[debuginfo-tests] Add support for moving debuginfo-tests from clang/test to
llvm/projects or monorepo.
Summary:
The new version of debuginfo-tests will have it's
Author: dhinton
Date: Tue Dec 12 11:47:40 2017
New Revision: 320519
URL: http://llvm.org/viewvc/llvm-project?rev=320519&view=rev
Log:
[cmake] Follow-up to rL320494.
EXISTS requires full paths.
Modified:
cfe/trunk/test/CMakeLists.txt
Modified: cfe/trunk/test/CMakeLists.txt
URL:
http://llvm.
Author: dhinton
Date: Thu Aug 3 09:13:13 2017
New Revision: 309955
URL: http://llvm.org/viewvc/llvm-project?rev=309955&view=rev
Log:
[diagtool] Add ability to pass in the id and return the name for a
particular diagnostic.
Differential Revision: https://reviews.llvm.org/D36252
Modified:
cfe
On Thu, Oct 26, 2017 at 3:00 PM, Greg Clayton via Phabricator <
revi...@reviews.llvm.org> wrote:
> clayborg added a comment.
>
> Each lldb.SBValue has accessors for the stuff in an execution context:
>
> ``
>
> lldb::SBTarget GetTarget();
> lldb::SBProcess GetProcess();
> lldb::SBThread GetT
On Thu, Oct 26, 2017 at 2:48 PM, Zachary Turner wrote:
> Seems fine, it would be nice if the workflow could be improved a little
> bit so that all you have to do is say `clangdiag break
> —error=“-Wcovered-switch”` or something . I think that gives the most
> intuitive usage for people, even it’s
On Thu, Oct 26, 2017 at 5:44 PM, Zachary Turner wrote:
>
>
> On Thu, Oct 26, 2017 at 3:18 PM Don Hinton wrote:
>
>> On Thu, Oct 26, 2017 at 2:48 PM, Zachary Turner
>> wrote:
>>
>>> Seems fine, it would be nice if the workflow could be improved a little
>>> bit so that all you have to do is say
Author: dhinton
Date: Fri Oct 27 10:02:33 2017
New Revision: 316773
URL: http://llvm.org/viewvc/llvm-project?rev=316773&view=rev
Log:
New lldb python module for managing diagnostic breakpoints
Summary:
Can be used to set breakpoints for either the diagnostics actually
emitted for the current comp
Hi Zachery:
I'm seeing a breakage when cross-compiling and including libcxx in the
runtimes directory.
The problem is that `include(AddLLVM)`, guarded by LIBCXX_INCLUDE_TESTS, comes
after your change to call `configure_lit_site_cfg`.
I've fixed this locally by adding `include(AddLLVM)` to the t
On Mon, Nov 20, 2017 at 5:20 PM, Zachary Turner via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: zturner
> Date: Mon Nov 20 17:20:28 2017
> New Revision: 318722
>
> URL: http://llvm.org/viewvc/llvm-project?rev=318722&view=rev
> Log:
> Re-revert "Refactor debuginfo-tests."
>
> This is
Minor nit: your diff is much bigger than it should have been due to
whitespace changes.
On Sun, Feb 5, 2017 at 9:21 AM, Saleem Abdulrasool via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> Author: compnerd
> Date: Sun Feb 5 11:21:52 2017
> New Revision: 294127
>
> URL: http://llvm.org/view
Author: dhinton
Date: Thu Jun 29 19:57:34 2017
New Revision: 306782
URL: http://llvm.org/viewvc/llvm-project?rev=306782&view=rev
Log:
Test Commit
Modified:
cfe/trunk/README.txt
Modified: cfe/trunk/README.txt
URL:
http://llvm.org/viewvc/llvm-project/cfe/trunk/README.txt?rev=306782&r1=306781&
Author: dhinton
Date: Thu Jun 29 20:03:28 2017
New Revision: 306783
URL: http://llvm.org/viewvc/llvm-project?rev=306783&view=rev
Log:
Remove test commit.
Modified:
cfe/trunk/README.txt
Modified: cfe/trunk/README.txt
URL:
http://llvm.org/viewvc/llvm-project/cfe/trunk/README.txt?rev=306783&r1
Author: dhinton
Date: Thu Jan 18 10:29:36 2018
New Revision: 322870
URL: http://llvm.org/viewvc/llvm-project?rev=322870&view=rev
Log:
[cmake] [libcxxabi] Don't print warning when tests are disabled.
Summary:
Don't print, possibly erroneous, warning if
LIBCXXABI_INCLUDE_TESTS is false.
This patch
Author: dhinton
Date: Fri Jan 19 10:31:12 2018
New Revision: 322977
URL: http://llvm.org/viewvc/llvm-project?rev=322977&view=rev
Log:
[cmake] Also pass CMAKE_ASM_COMPILER_ID to next stage when bootstrapping
Summary:
When setting CMAKE_ASM_COMPILER=clang, we also need to set
CMAKE_ASM_COMPILER_ID=
Author: dhinton
Date: Mon Jan 22 11:15:06 2018
New Revision: 323141
URL: http://llvm.org/viewvc/llvm-project?rev=323141&view=rev
Log:
[cmake] [libunwind] Fix find_path() problems when cross compiling
When CMAKE_SYSROOT or CMAKE_FIND_ROOT_PATH is set, cmake
recommends setting CMAKE_FIND_ROOT_PATH_
Author: dhinton
Date: Mon Jan 22 11:26:38 2018
New Revision: 323143
URL: http://llvm.org/viewvc/llvm-project?rev=323143&view=rev
Log:
[cmake] [libcxx] Fix find_path() problems when cross compiling.
When CMAKE_SYSROOT or CMAKE_FIND_ROOT_PATH is set, cmake
recommends setting CMAKE_FIND_ROOT_PATH_MO
Author: dhinton
Date: Mon Jan 22 11:41:05 2018
New Revision: 323145
URL: http://llvm.org/viewvc/llvm-project?rev=323145&view=rev
Log:
[cmake] [libcxxabi] Fix find_path() problems when cross compiling
When CMAKE_SYSROOT or CMAKE_FIND_ROOT_PATH is set, cmake
recommends setting CMAKE_FIND_ROOT_PATH_
Author: dhinton
Date: Mon Jan 22 19:30:23 2018
New Revision: 323171
URL: http://llvm.org/viewvc/llvm-project?rev=323171&view=rev
Log:
[cmake] Always respect existing CMAKE_REQUIRED_FLAGS when adding additional
ones.
* Previously part of https://reviews.llvm.org/D41622.
Modified:
libcxx/trun
Currently, herald doesn't subscribe libunwind to any group. Should it be
added to cfe-commits?
https://reviews.llvm.org/H268
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Thanks Ben...
On Thu, Jan 25, 2018 at 9:53 AM, Ben Hamilton
wrote:
> Ah, I see from http://bcain-llvm.readthedocs.io/projects/
> libunwind/en/latest/ that libunwind review discussion does take place on
> cfe-commits.
>
> Added. Sorry about not doing that earlier!
>
> Ben
>
> On Thu, Jan 25, 2018
Author: dhinton
Date: Thu Jan 25 10:13:26 2018
New Revision: 323453
URL: http://llvm.org/viewvc/llvm-project?rev=323453&view=rev
Log:
[cmake] [libcxx] Call llvm_setup_rpath() when adding shared libraries.
Clang and llvm already use llvm_setup_rpath(), so this change will
help standarize rpath usa
Author: dhinton
Date: Thu Jan 25 10:43:18 2018
New Revision: 323455
URL: http://llvm.org/viewvc/llvm-project?rev=323455&view=rev
Log:
[cmake] [libcxxabi] Call llvm_setup_rpath() when adding shared libraries.
Clang and llvm already use llvm_setup_rpath(), so this change will
help standarize rpath
Btw, I'm seeing other inconsistencies, e.g., I just committed
https://reviews.llvm.org/D42460 and herald ended up adding both cfe-commits
and llvm-commits to subscribers.
When I created it, I left subscribers blank, so herald added cfe-commits --
which seems to be correct.
Then, when I committed i
Author: dhinton
Date: Thu Jan 25 11:18:51 2018
New Revision: 323458
URL: http://llvm.org/viewvc/llvm-project?rev=323458&view=rev
Log:
Revert [libcxxabi] r323455 - [cmake] [libcxxabi] Call llvm_setup_rpath() when
adding shared libraries.
Shoaib Meenai pointed out this will break standalone builds
Author: dhinton
Date: Thu Jan 25 11:22:23 2018
New Revision: 323459
URL: http://llvm.org/viewvc/llvm-project?rev=323459&view=rev
Log:
Revert [libcxx] r323453 - [cmake] [libcxx] Call llvm_setup_rpath() when adding
shared libraries.
Shoaib Meenai pointed out this will break standalone builds when
Ah, I think that's the issue.
local:/Users/dhinton/projects/llvm_project/libcxxabi $ cat .arcconfig
{
"repository.callsign" : "CXXA",
"conduit_uri" : "https://reviews.llvm.org/";
}
So, it should set repository to CXXA, not LLVM.
On Thu, Jan 25, 2018 at 11:25 AM, Ben Hamilton
wrote:
> Yes
ository or even its CMake modules), right? Some upstream users care a
> lot about that use case (CC beanz and Duncan).
>
> *From: *cfe-commits on behalf of Don
> Hinton via cfe-commits
> *Reply-To: *Don Hinton
> *Date: *Thursday, January 25, 2018 at 10:15 AM
> *To: *"cfe
g libc++ standalone (i.e. without any LLVM
>> repository or even its CMake modules), right? Some upstream users care a
>> lot about that use case (CC beanz and Duncan).
>>
>> *From: *cfe-commits on behalf of
>> Don Hinton via cfe-commits
>> *Reply-To: *Don Hinton
modules along
>> side llvm-config, so if you find llvm-config, you find these modules.
>>
>> So, unless I'm missing something, it looks like AddLLVM.cmake is always
>> available.
>>
>>
>>>
>>> On Jan 25, 2018, at 11:02, Shoaib
Author: dhinton
Date: Thu Jan 25 17:34:51 2018
New Revision: 323492
URL: http://llvm.org/viewvc/llvm-project?rev=323492&view=rev
Log:
Reland: [cmake] [libcxx] Call llvm_setup_rpath() when adding shared libraries.
Clang and llvm already use llvm_setup_rpath(), so this change will
help standarize r
Author: dhinton
Date: Thu Jan 25 19:41:58 2018
New Revision: 323495
URL: http://llvm.org/viewvc/llvm-project?rev=323495&view=rev
Log:
Reland:
[cmake] [libcxxabi] Call llvm_setup_rpath() when adding shared libraries.
Clang and llvm already use llvm_setup_rpath(), so this change will
help standariz
Author: dhinton
Date: Thu Jan 25 20:01:50 2018
New Revision: 323496
URL: http://llvm.org/viewvc/llvm-project?rev=323496&view=rev
Log:
[cmake] [libunwind] Call llvm_setup_rpath() when adding shared libraries.
Clang and llvm already use llvm_setup_rpath(), so this change will
help standarize rpath
gt;>
>>>> That doesn't necessarily mean it's safe. Does anyone know why this
>>>> might have "worked"?
>>>>
>>>
>>> I'm not an expert, but llvm installs all of these cmake modules along
>>> side llvm-config, s
Author: dhinton
Date: Sat Jan 27 10:55:30 2018
New Revision: 323599
URL: http://llvm.org/viewvc/llvm-project?rev=323599&view=rev
Log:
LLVM_FOUND isn't always set, so just test if llvm_setup_rpath() is
available instead.
Modified:
libcxx/trunk/lib/CMakeLists.txt
Modified: libcxx/trunk/lib/CMa
Author: dhinton
Date: Sat Jan 27 11:18:04 2018
New Revision: 323600
URL: http://llvm.org/viewvc/llvm-project?rev=323600&view=rev
Log:
[cmake] [libcxxabi] LLVM_FOUND isn't always set, so just test if
llvm_setup_rpath() is available instead.
Modified:
libcxxabi/trunk/src/CMakeLists.txt
Modifie
Author: dhinton
Date: Sat Jan 27 11:31:44 2018
New Revision: 323601
URL: http://llvm.org/viewvc/llvm-project?rev=323601&view=rev
Log:
[cmake] [libunwind] LLVM_FOUND isn't always set, so just test if
llvm_setup_rpath() is available instead.
Modified:
libunwind/trunk/src/CMakeLists.txt
Modifie
hintonda added a comment.
In http://reviews.llvm.org/D12646#241308, @kubabrecka wrote:
> In what scenario exactly are you seeing an issue? If it's a just-built clang
> that can't find C++ headers, then you should just build the libcxx project
> alongside.
clang++ defaults to -stdlib=libc++ f
hintonda added a comment.
> I think it works if you a) check out libcxx into llvm/projects/libcxx b) run
> `make install` and c) run the binary from your install directory.
Yes, that works. But why shouldn't users be able to use the installed version?
Also, if you want to force users to eithe
hintonda added a comment.
In http://reviews.llvm.org/D12646#241493, @bob.wilson wrote:
> On Darwin platforms, the libc++ headers are expected to be installed
> alongside clang. If you're not doing that, then you're building it wrong.
> Adding more fallback options for finding the headers just m
hintonda added a comment.
I don't have commit access, so could you commit it for me?
thanks...
don
http://reviews.llvm.org/D12549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hintonda added a comment.
Looks like Manuel cleaned it up along with another change on the 8th, r247018.
thanks again...
http://reviews.llvm.org/D12549
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/li
hintonda created this revision.
hintonda added a reviewer: alexfh.
hintonda added a subscriber: cfe-commits.
Herald added a subscriber: klimek.
Fix example comments.
http://reviews.llvm.org/D12967
Files:
include/clang/Tooling/CommonOptionsParser.h
Index: include/clang/Tooling/CommonOptionsPar
hintonda added a comment.
Yes, please. thanks...
http://reviews.llvm.org/D12967
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hintonda created this revision.
hintonda added a subscriber: cfe-commits.
Add -Wc, option (similar to -Wl,) to go along with -Xclang. This
makes it easier to pass multiple options to cc1.
http://reviews.llvm.org/D13229
Files:
include/clang/Driver/Options.td
lib/Driver/Tools.cpp
Index: lib
n Mon, Sep 28, 2015 at 09:54:53PM +, don hinton via cfe-commits wrote:
> > Add -Wc, option (similar to -Wl,) to go along with -Xclang.
> > This makes it easier to pass multiple options to cc1.
>
> How is this better than -mllvm and -Xclang? There is no precendence case
>
hintonda added a comment.
Here are a few examples from existing tests showing how this option could be
used:
-Xclang -analyzer-max-loop -Xclang 34
becomes:
-Wc,-analyzer-max-loop,34
-Xclang -analyzer-checker=debug.ConfigDumper -Xclang -analyzer-max-loop
-Xclang 34
becomes:
-Wc,-a
hintonda added a comment.
Mainly aesthetic. It was the only one of the group, -Xassembler, -Xlinker,
-Xpreprocessor, and -Xclang, that didn't have a corresponding comma separated
option, so I figured it might be a good addition.
http://reviews.llvm.org/D13229
__
hintonda created this revision.
hintonda added a reviewer: chapuni.
hintonda added a subscriber: cfe-commits.
Always generate and install cmake config files.
Currently, cmake config files are only generated and installed when
CLANG_BUILD_STANDALONE set, which means config file will not be genera
gcc has -Wstrict-prototypes which will catch this, but clang doesn't
implement it.
however, both gcc and clang have -Wmissing-prototypes which should catch
these if users enable it.
On Sun, Aug 30, 2015 at 5:32 PM, Aaron Ballman via cfe-commits <
cfe-commits@lists.llvm.org> wrote:
> On Sun, Aug
hintonda created this revision.
hintonda added a reviewer: rsmith.
hintonda added a subscriber: cfe-commits.
Current behavior doesn't add c++ header path if libc++ wasn't installed via
macports in /usr. This change will try to locate c++ headers from various
locations, including in tree, via ma
hintonda updated this revision to Diff 34175.
hintonda removed reviewers: thakis, EricWF, kubabrecka.
hintonda added a comment.
Removed redundant code from InitHeaderSearch.cpp, it's handled more cleanly in
AddClangCXXStdlibIncludeArgs.
Also, newer versions of OSX add libc++ as the default. Per
Not sure why arc removed some reviewers when I added the latest diff, but
I've added them back.
Sorry for the noise...
On Mon, Sep 7, 2015 at 8:36 PM, don hinton wrote:
> hintonda updated this revision to Diff 34175.
> hintonda removed reviewers: thakis, EricWF, kubabrecka.
> hintonda added a c
hintonda added a comment.
Installed cmake 2.8.12 and was able to reproduce error.
I'll look into it, but I'm tempted to only support newer versions of cmake, and
let older versions maintain current behavior.
http://reviews.llvm.org/D13453
___
cfe-
hintonda updated this revision to Diff 36889.
hintonda added a comment.
- add back check for standalone and add cmake version
http://reviews.llvm.org/D13453
Files:
CMakeLists.txt
Index: CMakeLists.txt
===
--- CMakeLists.txt
+++
hintonda updated this revision to Diff 36916.
hintonda added a comment.
Add support for cmake version 3, i.e., 3.0.0...
http://reviews.llvm.org/D13453
Files:
CMakeLists.txt
Index: CMakeLists.txt
===
--- CMakeLists.txt
+++ CMakeL
hintonda added a comment.
Btw, I don't have commit access, so if you are happy with this change, could
you commit it for me?
http://reviews.llvm.org/D13453
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailma
hintonda created this revision.
hintonda added a reviewer: chapuni.
hintonda added a subscriber: cfe-commits.
ClangConfig requires LLVMConfig, so add find_package call in ClangConfig so
find_package(clang REQUIRED CONFIG) will just work. This makes it easier for
cmake based projects to use clan
hintonda created this revision.
hintonda added a reviewer: mclow.lists.
hintonda added a subscriber: cfe-commits.
http://reviews.llvm.org/D16946
Files:
www/get_started.html
Index: www/get_started.html
===
--- www/get_started.html
hintonda created this revision.
hintonda added reviewers: beanz, mclow.lists.
hintonda added a subscriber: cfe-commits.
A common error for new users it to checkout subprojects, like clang or
clang-tools-extra, into the wrong subdirectory. This change helps prevent
this, since the resulting buil
1 - 100 of 249 matches
Mail list logo
