jamesETsmith wrote:
@cjdb, thanks for taking a look! I've addressed some of your comments already
and will try to wrap up the rest early next week. I've got plenty of time to
work on this over the next couple of weeks so I shouldn't have a problem
wrapping this up before the end of January
ht
aeubanks wrote:
the file name is `vk_mem_alloc.h` so that shouldn't be the issue
https://github.com/llvm/llvm-project/pull/74809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2499,6 +2499,15 @@ TEST_F(TokenAnnotatorTest, BraceKind) {
EXPECT_BRACE_KIND(Tokens[6], BK_Block);
}
+TEST_F(TokenAnnotatorTest, StreamOperator) {
+ auto Tokens = annotate("\"foo\\n\" << aux << \"foo\\n\" << \"foo\";");
+ ASSERT_EQ(Tokens.size(), 9u) << Tokens;
+ EXPE
aeubanks wrote:
ah it's because we something like
```
[cfi-unrelated-cast|cfi-derived-cast]
src:*third_party/vulkan_memory_allocator/include/vk_mem_alloc.h
```
it seems like the new system doesn't match
`[cfi-unrelated-cast|cfi-derived-cast]`
https://github.com/llvm/llvm-project/pull/74809
_
https://github.com/XDeme updated https://github.com/llvm/llvm-project/pull/77045
>From d9cbbe48b96d27bff3fc926b60d039ed05f00489 Mon Sep 17 00:00:00 2001
From: XDeme
Date: Fri, 5 Jan 2024 01:23:16 -0300
Subject: [PATCH 1/4] [clang-format] Fix crash involving array designators and
dangling comma
@@ -1444,16 +1444,22 @@ WhitespaceManager::CellDescriptions
WhitespaceManager::getCells(unsigned Start,
} else if (C.Tok->is(tok::comma)) {
if (!Cells.empty())
Cells.back().EndIndex = i;
-if (C.Tok->getNextNonComment()->isNot(tok::r_brace)) // d
dyung wrote:
Hi @bwendling, your change to MemoryBuiltins.h is hitting an error in the
version of Visual Studio 2019 that we use internally to build:
```
C:\j\w\779ddbee\p\llvm\include\llvm/Analysis/MemoryBuiltins.h(217): error
C2990: 'llvm::SizeOffsetType': non-class template has already been
https://github.com/XDeme updated https://github.com/llvm/llvm-project/pull/77013
>From 45d01cbc7ec958518b1739daa9e9b0dc35c2d194 Mon Sep 17 00:00:00 2001
From: XDeme
Date: Thu, 4 Jan 2024 19:04:21 -0300
Subject: [PATCH 1/4] [clang-format] Handle templated elaborated type specifier
in function re
https://github.com/XDeme edited https://github.com/llvm/llvm-project/pull/77045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,333 @@
+//=- ClangDiagnosticsEmitter.cpp - Generate Clang diagnostics tables -*- C++
-*-
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/XDeme edited https://github.com/llvm/llvm-project/pull/77045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hanickadot updated
https://github.com/llvm/llvm-project/pull/76502
From b72e9299c957bd73744605e12d52b229fc289d5b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Hana=20Dusi=CC=81kova=CC=81?=
Date: Sun, 7 Jan 2024 00:13:08 +0100
Subject: [PATCH] [coverage] fix incorrect coverage repo
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Hana Dusíková (hanickadot)
Changes
Hi, I noticed some links in documentation is pointing to bad url (there were
two wrong paths)
I also noticed file `llvm/docs/tutorial/BuildingAJIT5.htm` is missing and it's
pointed from `BuildingAJIT4.h
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Hana Dusíková (hanickadot)
Changes
Hi, I noticed some links in documentation is pointing to bad url (there were
two wrong paths)
I also noticed file `llvm/docs/tutorial/BuildingAJIT5.htm` is missing and it's
pointed from `Buildin
https://github.com/HazardyKnusperkeks edited
https://github.com/llvm/llvm-project/pull/77045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1444,16 +1444,26 @@ WhitespaceManager::CellDescriptions
WhitespaceManager::getCells(unsigned Start,
} else if (C.Tok->is(tok::comma)) {
if (!Cells.empty())
Cells.back().EndIndex = i;
-if (C.Tok->getNextNonComment()->isNot(tok::r_brace)) // d
@@ -1444,16 +1444,26 @@ WhitespaceManager::CellDescriptions
WhitespaceManager::getCells(unsigned Start,
} else if (C.Tok->is(tok::comma)) {
if (!Cells.empty())
Cells.back().EndIndex = i;
-if (C.Tok->getNextNonComment()->isNot(tok::r_brace)) // d
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/77045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/77013
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hanickadot updated
https://github.com/llvm/llvm-project/pull/76502
From 93e8cc7c4310f8b91be4d72f4862f5ed7c06c33b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Hana=20Dusi=CC=81kova=CC=81?=
Date: Thu, 28 Dec 2023 14:30:35 +0100
Subject: [PATCH] [llvm] fix wrong links in documentati
https://github.com/hanickadot created
https://github.com/llvm/llvm-project/pull/77214
It was a while since I noticed coverage report is broken for "if constexpr" and
"if consteval" (as shown on first picture).
https://github.com/llvm/llvm-project/assets/6557263/dbdbc8a6-ad16-44da-882d-8e229ee69
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Hana Dusíková (hanickadot)
Changes
It was a while since I noticed coverage report is broken for "if constexpr" and
"if consteval" (as shown on first picture).
[clang] [clang][coverage] fixing "if constexpr" and "if consteval" coverage report (PR #77214)
https://github.com/hanickadot edited
https://github.com/llvm/llvm-project/pull/77214
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hanickadot wrote:
https://github.com/llvm/llvm-project/assets/6557263/47ff77c0-9101-44cf-b2d5-ffea514bfc0c";>
(notice wrong coverage "if constexpr" for positive one, and completely missing
for negative one, also notice "if consteval" marking always the same branch as
uncovered)
https://gith
https://github.com/XDeme updated https://github.com/llvm/llvm-project/pull/77045
>From d9cbbe48b96d27bff3fc926b60d039ed05f00489 Mon Sep 17 00:00:00 2001
From: XDeme
Date: Fri, 5 Jan 2024 01:23:16 -0300
Subject: [PATCH 1/5] [clang-format] Fix crash involving array designators and
dangling comma
https://github.com/hanickadot updated
https://github.com/llvm/llvm-project/pull/77214
From 8f1370aae4db2048c35516a85fb72c742557942b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Hana=20Dusi=CC=81kova=CC=81?=
Date: Sun, 7 Jan 2024 00:13:08 +0100
Subject: [PATCH] [coverage] fix incorrect coverage repo
https://github.com/hanickadot edited
https://github.com/llvm/llvm-project/pull/77214
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/trcrsired created
https://github.com/llvm/llvm-project/pull/77216
Particularly for canadian compilation
>From a1156a564c798192cb7608e28ce9b9ed846a8b1c Mon Sep 17 00:00:00 2001
From: trcrsired
Date: Sat, 6 Jan 2024 20:21:06 -0500
Subject: [PATCH] Fix Multiple Build Errors on
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be
notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it i
llvmbot wrote:
@llvm/pr-subscribers-lldb
Author: cqwrteur (trcrsired)
Changes
Particularly for canadian compilation
---
Patch is 23.86 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/77216.diff
8 Files Affected:
- (modified) clang-tools-extr
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: cqwrteur (trcrsired)
Changes
Particularly for canadian compilation
---
Patch is 23.86 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/77216.diff
8 Files Affected:
- (modified) clang-tools-ext
llvmbot wrote:
@llvm/pr-subscribers-mlir-execution-engine
Author: cqwrteur (trcrsired)
Changes
Particularly for canadian compilation
---
Patch is 23.86 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/77216.diff
8 Files Affected:
- (modified)
https://github.com/trcrsired updated
https://github.com/llvm/llvm-project/pull/77216
>From a1156a564c798192cb7608e28ce9b9ed846a8b1c Mon Sep 17 00:00:00 2001
From: trcrsired
Date: Sat, 6 Jan 2024 20:21:06 -0500
Subject: [PATCH 1/2] Fix Multiple Build Errors on different platforms
Particularly f
https://github.com/trcrsired updated
https://github.com/llvm/llvm-project/pull/77216
>From a1156a564c798192cb7608e28ce9b9ed846a8b1c Mon Sep 17 00:00:00 2001
From: trcrsired
Date: Sat, 6 Jan 2024 20:21:06 -0500
Subject: [PATCH 1/2] Fix Multiple Build Errors on different platforms
Particularly f
@@ -46,64 +46,64 @@ class Triple {
enum ArchType {
UnknownArch,
-arm,// ARM (little endian): arm, armv.*, xscale
asl wrote:
Please do not do such large code reformatting. It makes impossible to review
them
https://github.com/llvm/llvm-
@@ -108,13 +108,9 @@ else()
linux/HostInfoLinux.cpp
linux/LibcGlue.cpp
linux/Support.cpp
+ android/HostInfoAndroid.cpp
+ android/LibcGlue.cpp
)
-if (CMAKE_SYSTEM_NAME MATCHES "Android")
asl wrote:
What is wrong with andro
@@ -108,13 +108,9 @@ else()
linux/HostInfoLinux.cpp
linux/LibcGlue.cpp
linux/Support.cpp
+ android/HostInfoAndroid.cpp
+ android/LibcGlue.cpp
)
-if (CMAKE_SYSTEM_NAME MATCHES "Android")
trcrsired wrote:
I use "Linux" to d
@@ -46,64 +46,64 @@ class Triple {
enum ArchType {
UnknownArch,
-arm,// ARM (little endian): arm, armv.*, xscale
trcrsired wrote:
without doing code formatting the CI won't pass.
https://github.com/llvm/llvm-project/pull/77216
_
@@ -46,64 +46,64 @@ class Triple {
enum ArchType {
UnknownArch,
-arm,// ARM (little endian): arm, armv.*, xscale
asl wrote:
You can ignore these.
https://github.com/llvm/llvm-project/pull/77216
__
https://github.com/asl edited https://github.com/llvm/llvm-project/pull/77216
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -108,13 +108,9 @@ else()
linux/HostInfoLinux.cpp
linux/LibcGlue.cpp
linux/Support.cpp
+ android/HostInfoAndroid.cpp
+ android/LibcGlue.cpp
)
-if (CMAKE_SYSTEM_NAME MATCHES "Android")
asl wrote:
Well, so this is not a c
@@ -46,64 +46,64 @@ class Triple {
enum ArchType {
UnknownArch,
-arm,// ARM (little endian): arm, armv.*, xscale
trcrsired wrote:
i will fix it later. but first pass CI with other things first to ensure it
does not break anything.
http
@@ -108,13 +108,9 @@ else()
linux/HostInfoLinux.cpp
linux/LibcGlue.cpp
linux/Support.cpp
+ android/HostInfoAndroid.cpp
+ android/LibcGlue.cpp
)
-if (CMAKE_SYSTEM_NAME MATCHES "Android")
trcrsired wrote:
> Well, so this is
@@ -108,13 +108,9 @@ else()
linux/HostInfoLinux.cpp
linux/LibcGlue.cpp
linux/Support.cpp
+ android/HostInfoAndroid.cpp
+ android/LibcGlue.cpp
)
-if (CMAKE_SYSTEM_NAME MATCHES "Android")
asl wrote:
Not quite. It's distant
srcarroll wrote:
I made a PR to extend UB cases in verifier
https://github.com/llvm/llvm-project/pull/77217
https://github.com/llvm/llvm-project/pull/76003
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/
@@ -108,13 +108,9 @@ else()
linux/HostInfoLinux.cpp
linux/LibcGlue.cpp
linux/Support.cpp
+ android/HostInfoAndroid.cpp
+ android/LibcGlue.cpp
)
-if (CMAKE_SYSTEM_NAME MATCHES "Android")
trcrsired wrote:
That is probably b
@@ -108,13 +108,9 @@ else()
linux/HostInfoLinux.cpp
linux/LibcGlue.cpp
linux/Support.cpp
+ android/HostInfoAndroid.cpp
+ android/LibcGlue.cpp
)
-if (CMAKE_SYSTEM_NAME MATCHES "Android")
trcrsired wrote:
I will try to find
https://github.com/JDevlieghere edited
https://github.com/llvm/llvm-project/pull/77216
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -108,13 +108,9 @@ else()
linux/HostInfoLinux.cpp
linux/LibcGlue.cpp
linux/Support.cpp
+ android/HostInfoAndroid.cpp
+ android/LibcGlue.cpp
)
-if (CMAKE_SYSTEM_NAME MATCHES "Android")
JDevlieghere wrote:
While Android m
https://github.com/JDevlieghere requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/77216
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sott0n wrote:
@kuhar @antiagainst Thanks for your comment and support to land this PR! If you
have any issues or TODO tasks in SPIR-V dialect, I would be happy if you could
share it with me.
https://github.com/llvm/llvm-project/pull/76353
___
cfe-com
https://github.com/zeroomega commented:
I test run it with C++26 on Windows, it clears the test failures happened in
the original PR.
https://github.com/llvm/llvm-project/pull/77190
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists
antiagainst wrote:
> @kuhar @antiagainst Thanks for your comment and support to land this PR! If
> you have any issues or TODO tasks in SPIR-V dialect, I would be happy if you
> could share it with me. :)
Thanks for your further interest! Please feel free to take a look at issues
with the `ml
https://github.com/Da-Viper updated
https://github.com/llvm/llvm-project/pull/76117
>From 97eeda4684804229d9faad19ea7b7888dcd91786 Mon Sep 17 00:00:00 2001
From: Ezike Ebuka
Date: Thu, 21 Dec 2023 02:30:34 +
Subject: [PATCH 1/6] Fix #75686: add iter_swap and iter_move to the matched
name
MaskRay wrote:
> ah it's because we something like
>
> ```
> [cfi-unrelated-cast|cfi-derived-cast]
>
> src:*third_party/vulkan_memory_allocator/include/vk_mem_alloc.h
> ```
>
> it seems like the new system doesn't match
> `[cfi-unrelated-cast|cfi-derived-cast]`
The glob mode can use the sect
https://github.com/linux4life798 created
https://github.com/llvm/llvm-project/pull/77228
Remove Python 2 support and clean up code that conditions based on version.
Issue #76664.
>From 1ce6508ee1052c46c44a0f8f5dfd424df83ce209 Mon Sep 17 00:00:00 2001
From: Craig Hesling
Date: Sat, 2 Dec 2023
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Craig Hesling (linux4life798)
Changes
Remove Python 2 support and clean up code that conditions based on version.
Issue #76664.
---
Full diff: https://github.com/llvm/llvm-project/pull/77228.diff
2 Files Affected:
- (modified) clang/bi
101 - 157 of 157 matches
Mail list logo