[clang] [lldb] [llvm] [lldb][test] Remove LLDB_TEST_USE_VENDOR_PACKAGES (PR #89260)

2024-04-18 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht closed https://github.com/llvm/llvm-project/pull/89260 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [lldb] [llvm] [lldb][test] Remove LLDB_TEST_USE_VENDOR_PACKAGES (PR #89260)

2024-04-18 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht created https://github.com/llvm/llvm-project/pull/89260 The `LLDB_TEST_USE_VENDOR_PACKAGES` has defaulted to `Off` for a while. Either installing `pexpect` or skipping those tests with `-DLLDB_TEST_USER_ARGS=--skip-category=pexpect` seems to be enough that we can

[clang] [llvm] [clang] Migrate clang-rename to OptTable parsing (PR #89167)

2024-04-17 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht created https://github.com/llvm/llvm-project/pull/89167 Using OptTable to parse will allow including this tool in llvm-driver. Because CommonOptionsParser is widely used and makes use of `cl::opt` flags, it needs to be refactored to handle both. The existing

[clang] Reapply "[Clang][Sema] Fix crash when 'this' is used in a dependent class scope function template specialization that instantiates to a static member function (#87541)" (PR #88311)

2024-04-12 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: Still seeing a few more failures, although the original repro is now working. ```c++ struct Base { int BaseFunc(int x) { return 0; } int val; }; template struct Foo : Base { template int bar(X x) { return BaseFunc(val); // OK } template <>

[clang] [Flang] responds to Clang Tidy feedback (PR #87847)

2024-04-10 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht approved this pull request. https://github.com/llvm/llvm-project/pull/87847 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Flang] responds to Clang Tidy feedback (PR #87847)

2024-04-10 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht edited https://github.com/llvm/llvm-project/pull/87847 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] responds to Clang Tidy feedback (PR #87847)

2024-04-10 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht edited https://github.com/llvm/llvm-project/pull/87847 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][test] Avoid writing to a potentially write-protected dir (PR #88258)

2024-04-10 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht closed https://github.com/llvm/llvm-project/pull/88258 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][test] Avoid writing to a potentially write-protected dir (PR #88258)

2024-04-10 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht created https://github.com/llvm/llvm-project/pull/88258 This test just checks for the stdout/stderr of clang, but it incidentally tries to write to `a.out` in the current directory, which may be write protected. Typically one would write `clang -o %t.o` for a

[clang] [Clang][Sema] Fix crash when 'this' is used in a dependent class scope function template specialization that instantiates to a static member function (PR #87541)

2024-04-09 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: This commit appears to regress an example like this: ```c++ template struct Foo { template int bar(X x) { return 0; } template <> int bar(int x) { return bar(5.0); } }; void call() { Foo f; f.bar(1); } ``` Used to compile, now results in an error:

[clang-tools-extra] [llvm] [bazel] Add clangd as a library support (PR #81556)

2024-03-14 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: Thanks! Sorry about the delay, we should get the bzl tree setup for automatic assignment to reviewers. https://github.com/llvm/llvm-project/pull/81556 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang-tools-extra] [llvm] [bazel] Add clangd as a library support (PR #81556)

2024-03-14 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht closed https://github.com/llvm/llvm-project/pull/81556 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [llvm] [bazel] Add clangd as a library support (PR #81556)

2024-03-14 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht approved this pull request. https://github.com/llvm/llvm-project/pull/81556 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [llvm] Add bazel support for clangd as a library. (PR #81556)

2024-03-14 Thread Jordan Rupprecht via cfe-commits
@@ -0,0 +1,45 @@ +# This file is licensed under the Apache License v2.0 with LLVM Exceptions. +# See https://llvm.org/LICENSE.txt for license information. +# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception + +package(default_visibility = ["//visibility:public"]) +

[clang-tools-extra] [llvm] Add bazel support for clangd as a library. (PR #81556)

2024-03-14 Thread Jordan Rupprecht via cfe-commits
@@ -0,0 +1,45 @@ +# This file is licensed under the Apache License v2.0 with LLVM Exceptions. +# See https://llvm.org/LICENSE.txt for license information. +# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception + +package(default_visibility = ["//visibility:public"]) +

[clang] [Clang][Sema] Diagnose use of template keyword after declarative nested-name-specifiers (PR #78595)

2024-02-07 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: Ok, thanks for the quick confirmation! https://github.com/llvm/llvm-project/pull/78595 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Sema] Diagnose use of template keyword after declarative nested-name-specifiers (PR #78595)

2024-02-07 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: It looks like this caused some new `-Wextra-qualification` warnings (playground: https://godbolt.org/z/3MbMjGYET) ``` namespace foo { template struct Z {}; template <> struct Z {}; // OK template <> struct foo::Z {}; // New warning: extra qualification on member 'Z'

[clang-tools-extra] 96ec447 - [NFC] Fix various unintentional `//namespace` formatting

2024-01-24 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2024-01-24T21:17:45-08:00 New Revision: 96ec447a6ae39edff92674e9849de2bf21bfd430 URL: https://github.com/llvm/llvm-project/commit/96ec447a6ae39edff92674e9849de2bf21bfd430 DIFF:

[clang] 96ec447 - [NFC] Fix various unintentional `//namespace` formatting

2024-01-24 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2024-01-24T21:17:45-08:00 New Revision: 96ec447a6ae39edff92674e9849de2bf21bfd430 URL: https://github.com/llvm/llvm-project/commit/96ec447a6ae39edff92674e9849de2bf21bfd430 DIFF:

[llvm] [clang-tools-extra] [clang] [libc] [mlir] [lldb] [lldb][test] Apply @expectedFailureAll/@skipIf early for debug_info tests (PR #73067)

2024-01-19 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht closed https://github.com/llvm/llvm-project/pull/73067 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [lldb] [llvm] [clang] [lldb][test] Apply @expectedFailureAll/@skipIf early for debug_info tests (PR #73067)

2024-01-19 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht updated https://github.com/llvm/llvm-project/pull/73067 >From 22bfc5878f1f96b3138a03eea4dc856948185c89 Mon Sep 17 00:00:00 2001 From: Jordan Rupprecht Date: Tue, 21 Nov 2023 17:28:30 -0800 Subject: [PATCH 1/2] [lldb][test] Apply @expectedFailureAll/@skipIf early

[clang] [clang] Substitute alias templates from correct context (PR #75069)

2023-12-14 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: Thanks to @MaskRay for handling the revert. A reduction for the second `typename` breakage looks similar, so may be the same underlying issue. But here it is anyway, in case it ends up being a second bug in the patch: ```c++ class Foo {}; template struct Ptr {}; template

[clang] [clang] Substitute alias templates from correct context (PR #75069)

2023-12-14 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: After this commit, I'm seeing another "out-of-line definition" error for `operator=` on the following reduced snippet: ```c++ class Foo {}; template struct Stuff; template struct Stuff { Stuff& operator=(Stuff&& that); }; template using Alias = Stuff; template Alias&

[clang-tools-extra] [lldb] [llvm] [clang] [lldb][test] Apply @expectedFailureAll/@skipIf early for debug_info tests (PR #73067)

2023-12-05 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: > Is this a performance optimization or a function al change? Neither. This should only affect tests, and the set of tests we skip/mark as xfail should not change, we just generally know about it earlier in the test setup. https://github.com/llvm/llvm-project/pull/73067

[clang] [clang-tools-extra] [llvm] [lldb] [lldb][test] Apply @expectedFailureAll/@skipIf early for debug_info tests (PR #73067)

2023-12-05 Thread Jordan Rupprecht via cfe-commits
rupprecht wrote: ping :) https://github.com/llvm/llvm-project/pull/73067 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[llvm] [clang-tools-extra] [clang] [lldb] [lldb][test] Apply @expectedFailureAll/@skipIf early for debug_info tests (PR #73067)

2023-11-28 Thread Jordan Rupprecht via cfe-commits
https://github.com/rupprecht updated https://github.com/llvm/llvm-project/pull/73067 >From 22bfc5878f1f96b3138a03eea4dc856948185c89 Mon Sep 17 00:00:00 2001 From: Jordan Rupprecht Date: Tue, 21 Nov 2023 17:28:30 -0800 Subject: [PATCH 1/2] [lldb][test] Apply @expectedFailureAll/@skipIf early

[clang] 8b39527 - [NFC] Wrap entire debug logging loop in LLVM_DEBUG

2023-04-26 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2023-04-26T05:28:15-07:00 New Revision: 8b39527535ff50837da08509392e3268fb6129f0 URL: https://github.com/llvm/llvm-project/commit/8b39527535ff50837da08509392e3268fb6129f0 DIFF:

[clang] 74ce297 - Revert "[Clang] Implement Change scope of lambda trailing-return-type"

2023-02-03 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2023-02-03T08:49:34-08:00 New Revision: 74ce297045bac4bc475b8e762d2a1ea19bb16d3c URL: https://github.com/llvm/llvm-project/commit/74ce297045bac4bc475b8e762d2a1ea19bb16d3c DIFF:

[clang] 3432f4b - [test] Split out Annotations from `TestingSupport`

2023-01-12 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2023-01-12T13:40:47-08:00 New Revision: 3432f4bf86e7c77666ae9dede1610ae843dde648 URL: https://github.com/llvm/llvm-project/commit/3432f4bf86e7c77666ae9dede1610ae843dde648 DIFF:

[clang] 5a06334 - [Format] Capture `FormatStyle` by value to avoid use-after-free.

2022-12-15 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2022-12-15T20:38:32-08:00 New Revision: 5a06334c51aa75d7f044785a495cf2de5bf13a9c URL: https://github.com/llvm/llvm-project/commit/5a06334c51aa75d7f044785a495cf2de5bf13a9c DIFF:

[clang] fc7b8e7 - [test] Fix dr324.c again for non-writeable source directories

2022-12-12 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2022-12-12T10:11:19-08:00 New Revision: fc7b8e71312f89986be1a7cc67377a2decfcf337 URL: https://github.com/llvm/llvm-project/commit/fc7b8e71312f89986be1a7cc67377a2decfcf337 DIFF:

[clang] 493509a - [NFC] DeclCXX: Fix -Wreorder-ctor

2022-01-25 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2022-01-25T14:29:35-08:00 New Revision: 493509a40ad1653feb779861badd3ee59fc4a79c URL: https://github.com/llvm/llvm-project/commit/493509a40ad1653feb779861badd3ee59fc4a79c DIFF:

[clang] 1a368ae - [CUDA] fix builtin constraints for PTX 7.2

2021-02-19 Thread Jordan Rupprecht via cfe-commits
Author: Artem Belevich Date: 2021-02-19T09:57:21-08:00 New Revision: 1a368ae3b78dd7a364e8f17658fddaf86b1e98db URL: https://github.com/llvm/llvm-project/commit/1a368ae3b78dd7a364e8f17658fddaf86b1e98db DIFF:

[clang] 60a8a50 - [llvm-objdump] Print file format in lowercase to match GNU output.

2020-02-12 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2020-02-12T08:17:01-08:00 New Revision: 60a8a504f16dbbc5f2a6887ecb668ef4cb834949 URL: https://github.com/llvm/llvm-project/commit/60a8a504f16dbbc5f2a6887ecb668ef4cb834949 DIFF:

[clang] fafddbd - Revert "[Clang][Driver] Remove -M group options ..." and "[Clang] Avoid crashing when generating crash diagnostics when '#pragma clang __debug ..."

2020-02-06 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2020-02-06T17:59:15-08:00 New Revision: fafddbd956dbe439787f6d717c247e648bb07ff5 URL: https://github.com/llvm/llvm-project/commit/fafddbd956dbe439787f6d717c247e648bb07ff5 DIFF:

[clang] 553a727 - [clang] Remove -Wexperimental-float-control.

2019-12-18 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2019-12-18T16:51:55-08:00 New Revision: 553a727f5f6407fb6db7ac2dae5f5b2a536d38fc URL: https://github.com/llvm/llvm-project/commit/553a727f5f6407fb6db7ac2dae5f5b2a536d38fc DIFF:

Re: r374288 - Recommit "[Clang] Pragma vectorize_width() implies vectorize(enable)"

2019-10-24 Thread Jordan Rupprecht via cfe-commits
Reverted in 6d424a161bf3e52730371da0b9439ed93a8ce406 due to the issue described here. Should hopefully be a trivial fix forward. On Tue, Oct 22, 2019 at 2:46 PM Michael Kruse wrote: > Am Mo., 21. Okt. 2019 um 23:44 Uhr schrieb Jordan Rupprecht > : > > At any rate, it sounds like this is not a

[clang] 6d424a1 - Revert "Recommit "[Clang] Pragma vectorize_width() implies vectorize(enable)""

2019-10-24 Thread Jordan Rupprecht via cfe-commits
Author: Jordan Rupprecht Date: 2019-10-24T16:35:45-07:00 New Revision: 6d424a161bf3e52730371da0b9439ed93a8ce406 URL: https://github.com/llvm/llvm-project/commit/6d424a161bf3e52730371da0b9439ed93a8ce406 DIFF:

Re: r374288 - Recommit "[Clang] Pragma vectorize_width() implies vectorize(enable)"

2019-10-21 Thread Jordan Rupprecht via cfe-commits
On Mon, Oct 21, 2019 at 5:12 PM Michael Kruse wrote: > Am Mo., 21. Okt. 2019 um 16:01 Uhr schrieb Jordan Rupprecht via > cfe-commits : > > There's also a curious failure caused by this patch (confirmed passing > @r374287, failing @r374288): > > It's a warning, not a fail

Re: r374288 - Recommit "[Clang] Pragma vectorize_width() implies vectorize(enable)"

2019-10-21 Thread Jordan Rupprecht via cfe-commits
There's also a curious failure caused by this patch (confirmed passing @r374287, failing @r374288): $ cat /tmp/vectorize.cc void a() { #pragma clang loop vectorize(disable) for (;;) ; } $ clang++ -Werror -O3 -c /tmp/vectorize.cc /tmp/vectorize.cc:1:6: error: loop not interleaved: the

r373400 - [clang][OpenMP][NFC] #include GlobalDecl.h to avoid incomplete class type

2019-10-01 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Tue Oct 1 15:30:10 2019 New Revision: 373400 URL: http://llvm.org/viewvc/llvm-project?rev=373400=rev Log: [clang][OpenMP][NFC] #include GlobalDecl.h to avoid incomplete class type Modified: cfe/trunk/lib/CodeGen/CGOpenMPRuntime.h Modified:

r367256 - [driver][test] Use /dev/null in as-options.s instead

2019-07-29 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Mon Jul 29 13:09:20 2019 New Revision: 367256 URL: http://llvm.org/viewvc/llvm-project?rev=367256=rev Log: [driver][test] Use /dev/null in as-options.s instead Modified: cfe/trunk/test/Driver/as-options.s Modified: cfe/trunk/test/Driver/as-options.s URL:

r367253 - [driver][test] Update as-options.s to not write to a readonly tree

2019-07-29 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Mon Jul 29 12:57:31 2019 New Revision: 367253 URL: http://llvm.org/viewvc/llvm-project?rev=367253=rev Log: [driver][test] Update as-options.s to not write to a readonly tree The as-options.s test writes to the build tree as of r367165. Some build systems configure this

r365850 - Re-Revert Devirtualize destructor of final class.

2019-07-11 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Thu Jul 11 17:32:08 2019 New Revision: 365850 URL: http://llvm.org/viewvc/llvm-project?rev=365850=rev Log: Re-Revert Devirtualize destructor of final class. This reverts r365509 (git commit d088720edad9c29ee0d622b5d69092e18a9ac0bd) This is a second revert[1] due to

r363996 - [CodeGen][test] Use FileCheck variable matchers for better test support

2019-06-20 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Thu Jun 20 15:35:52 2019 New Revision: 363996 URL: http://llvm.org/viewvc/llvm-project?rev=363996=rev Log: [CodeGen][test] Use FileCheck variable matchers for better test support Summary: Depending on how clang is built, it may discard the IR names and use names like

r363848 - [clang][test] Add missing LambdaTemplateParams test and migrate from getLocStart

2019-06-19 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Wed Jun 19 10:43:58 2019 New Revision: 363848 URL: http://llvm.org/viewvc/llvm-project?rev=363848=rev Log: [clang][test] Add missing LambdaTemplateParams test and migrate from getLocStart These were removed a long time ago in r341573, but this test was missed because it

r360086 - Revert "Re-commit r357452: SimplifyCFG SinkCommonCodeFromPredecessors: Also sink function calls without used results (PR41259)"

2019-05-06 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Mon May 6 14:55:05 2019 New Revision: 360086 URL: http://llvm.org/viewvc/llvm-project?rev=360086=rev Log: Revert "Re-commit r357452: SimplifyCFG SinkCommonCodeFromPredecessors: Also sink function calls without used results (PR41259)" This reverts r357452 (git commit

r356700 - [clang][OpenMP] Fix another test when using libgomp.

2019-03-21 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Thu Mar 21 12:13:22 2019 New Revision: 356700 URL: http://llvm.org/viewvc/llvm-project?rev=356700=rev Log: [clang][OpenMP] Fix another test when using libgomp. Similarly to r356614, -fopenmp=libomp needs to be used for some omp-related AST matching. Modified:

[clang-tools-extra] r356623 - Revert "[clangd] Print arguments in template specializations"

2019-03-20 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Wed Mar 20 15:51:56 2019 New Revision: 356623 URL: http://llvm.org/viewvc/llvm-project?rev=356623=rev Log: Revert "[clangd] Print arguments in template specializations" This reverts commit 44a63f6a150dec72dea43730d2a89d292e58bd6f. It segfaults on an internal test case

r356623 - Revert "[clangd] Print arguments in template specializations"

2019-03-20 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Wed Mar 20 15:51:56 2019 New Revision: 356623 URL: http://llvm.org/viewvc/llvm-project?rev=356623=rev Log: Revert "[clangd] Print arguments in template specializations" This reverts commit 44a63f6a150dec72dea43730d2a89d292e58bd6f. It segfaults on an internal test case

r356614 - [clang][OpenMP] Fix build when using libgomp

2019-03-20 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Wed Mar 20 14:01:56 2019 New Revision: 356614 URL: http://llvm.org/viewvc/llvm-project?rev=356614=rev Log: [clang][OpenMP] Fix build when using libgomp Summary: rL356570 introduced a test which only passes with the default openmp library, libomp, and fails with other

r356517 - Fix CodeGen/arm64-microsoft-status-reg.cpp test

2019-03-19 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Tue Mar 19 13:55:14 2019 New Revision: 356517 URL: http://llvm.org/viewvc/llvm-project?rev=356517=rev Log: Fix CodeGen/arm64-microsoft-status-reg.cpp test Summary: This test is failing after r356499 (verified with `ninja check-clang-codegen`). Update the register

r356099 - [clang-format] Propagate inferred language to getLLVMStyle() in getPredefinedStyle()

2019-03-13 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Wed Mar 13 14:13:01 2019 New Revision: 356099 URL: http://llvm.org/viewvc/llvm-project?rev=356099=rev Log: [clang-format] Propagate inferred language to getLLVMStyle() in getPredefinedStyle() rC355158 added an optional language parameter to getLLVMStyle(), but this

r356097 - [clang-format][NFC] Include TableGen in enum->string mapping used for debugging

2019-03-13 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Wed Mar 13 13:34:34 2019 New Revision: 356097 URL: http://llvm.org/viewvc/llvm-project?rev=356097=rev Log: [clang-format][NFC] Include TableGen in enum->string mapping used for debugging Running `clang-format -debug` prints "Unknown" for tablegen files because of this

r355158 - [clang-format][TableGen] Don't add spaces around items in square braces.

2019-02-28 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Thu Feb 28 16:12:18 2019 New Revision: 355158 URL: http://llvm.org/viewvc/llvm-project?rev=355158=rev Log: [clang-format][TableGen] Don't add spaces around items in square braces. Summary: clang-formatting wants to add spaces around items in square braces, e.g. [1, 2] ->

r355123 - [clang-format][NFC] Allow getLLVMStyle() to take a language

2019-02-28 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Thu Feb 28 11:16:45 2019 New Revision: 355123 URL: http://llvm.org/viewvc/llvm-project?rev=355123=rev Log: [clang-format][NFC] Allow getLLVMStyle() to take a language Summary: getLLVMStyle() sets the default style, but doesn't take the language as a parameter, so can't

Re: r351391 - Recommit r351160 "[X86] Make _xgetbv/_xsetbv on non-windows platforms"

2019-01-18 Thread Jordan Rupprecht via cfe-commits
Confirmed that opencv builds now, thanks :) On Fri, Jan 18, 2019 at 10:05 AM Jordan Rupprecht wrote: > Thanks! I'll rebuild clang from trunk and give it another try. > Yes, for the second two, I think it should be a more straightforward fix a > la

Re: r351391 - Recommit r351160 "[X86] Make _xgetbv/_xsetbv on non-windows platforms"

2019-01-18 Thread Jordan Rupprecht via cfe-commits
Thanks! I'll rebuild clang from trunk and give it another try. Yes, for the second two, I think it should be a more straightforward fix a la https://chromium-review.googlesource.com/c/v8/v8/+/1414858 On Fri, Jan 18, 2019 at 9:58 AM Topper, Craig wrote: > I just committed a change to go back to

Re: r351391 - Recommit r351160 "[X86] Make _xgetbv/_xsetbv on non-windows platforms"

2019-01-18 Thread Jordan Rupprecht via cfe-commits
Hi Craig, We're seeing issues building a few other things now; notably this part in opencv: https://github.com/opencv/opencv/blob/master/modules/core/src/system.cpp#L422 #ifdef _XCR_XFEATURE_ENABLED_MASK // requires immintrin.h xcr0 =

r346687 - [clang-format] Support breaking consecutive string literals for TableGen

2018-11-12 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Mon Nov 12 10:15:04 2018 New Revision: 346687 URL: http://llvm.org/viewvc/llvm-project?rev=346687=rev Log: [clang-format] Support breaking consecutive string literals for TableGen Summary: clang-format can get confused by string literals in TableGen: it knows that

r343744 - [llvm-nm] Update clang tests based on changed llvm-nm output

2018-10-03 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Wed Oct 3 16:40:04 2018 New Revision: 343744 URL: http://llvm.org/viewvc/llvm-project?rev=343744=rev Log: [llvm-nm] Update clang tests based on changed llvm-nm output Modified: cfe/trunk/test/CodeGen/thinlto_backend.ll Modified:

Re: r342053 - [CodeGen] Align rtti and vtable data

2018-09-17 Thread Jordan Rupprecht via cfe-commits
> Interesting, what kind of failures? > > If they are causing you problems, of course feel free to revert. > > Dave Turns out they are all real issues which running the test under asan mode flags as global-buffer-overflow. I'm guessing the over-alignment was hiding the bug as reads there would be

r341002 - [AttrDocs] Fix build bots: add missing GNUInline pragma to test.

2018-08-29 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Wed Aug 29 17:04:34 2018 New Revision: 341002 URL: http://llvm.org/viewvc/llvm-project?rev=341002=rev Log: [AttrDocs] Fix build bots: add missing GNUInline pragma to test. Modified: cfe/trunk/test/Misc/pragma-attribute-supported-attributes-list.test Modified:

[libunwind] r339899 - NFC: Test commit access

2018-08-16 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Thu Aug 16 09:55:07 2018 New Revision: 339899 URL: http://llvm.org/viewvc/llvm-project?rev=339899=rev Log: NFC: Test commit access Testing commit access from a new machine, so using this as an opportunity to revert my old test access commit (r336006) that I never cleaned

[libcxx] r338157 - [libc++] Exclude posix_l/strtonum fallback inclusion for newlib > 2.4

2018-07-27 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Fri Jul 27 13:02:58 2018 New Revision: 338157 URL: http://llvm.org/viewvc/llvm-project?rev=338157=rev Log: [libc++] Exclude posix_l/strtonum fallback inclusion for newlib > 2.4 Summary: [libc++] Exclude posix_l/strtonum fallback inclusion for newlib > 2.4 r338122 changed

Re: r337850 - Revert "[VFS] Cleanups to VFS interfaces."

2018-07-25 Thread Jordan Rupprecht via cfe-commits
s.cpp:953: Failure Value of: *Locations Expected: has 1 element that is equal to 0:0-0:0@file:///clangd-test/foo.h Actual: {} > > On Tue, Jul 24, 2018 at 10:28 PM Jordan Rupprecht via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Author: rupprecht >> Date: T

r337850 - Revert "[VFS] Cleanups to VFS interfaces."

2018-07-24 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Tue Jul 24 13:28:07 2018 New Revision: 337850 URL: http://llvm.org/viewvc/llvm-project?rev=337850=rev Log: Revert "[VFS] Cleanups to VFS interfaces." This reverts commit r337834 due to test failures. Modified: cfe/trunk/include/clang/Basic/VirtualFileSystem.h

[libunwind] r336014 - Introduce a separate preprocessor macro, _LIBUNWIND_USE_DLADDR, for directly controlling a dependency on dladdr(). This will allow us to use libunwind without adding a libdl depe

2018-06-29 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Fri Jun 29 13:41:50 2018 New Revision: 336014 URL: http://llvm.org/viewvc/llvm-project?rev=336014=rev Log: Introduce a separate preprocessor macro, _LIBUNWIND_USE_DLADDR, for directly controlling a dependency on dladdr(). This will allow us to use libunwind without

[libunwind] r336006 - Add a blank line to docs/README.txt test commit access

2018-06-29 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Fri Jun 29 12:05:21 2018 New Revision: 336006 URL: http://llvm.org/viewvc/llvm-project?rev=336006=rev Log: Add a blank line to docs/README.txt test commit access Subscribers: christof, cfe-commits Differential Revision: https://reviews.llvm.org/D48785 Modified:

[libunwind] r336005 - Revert "Support for multiarch runtimes layout"

2018-06-29 Thread Jordan Rupprecht via cfe-commits
Author: rupprecht Date: Fri Jun 29 12:05:20 2018 New Revision: 336005 URL: http://llvm.org/viewvc/llvm-project?rev=336005=rev Log: Revert "Support for multiarch runtimes layout" This reverts commit 0c7cea3c0c6338b99e30c13201365a3dd4edc6f4. Modified: libunwind/trunk/CMakeLists.txt Modified: