https://github.com/albus-droid updated
https://github.com/llvm/llvm-project/pull/139986
>From 3cc8334092853442f85c5a17a3bd31e373f30da8 Mon Sep 17 00:00:00 2001
From: albus-droid
Date: Wed, 14 May 2025 15:49:11 -0400
Subject: [PATCH 1/4] Added two conditions to check for width of the factor
var
@@ -355,6 +349,9 @@ function(add_link_opts target_name)
set_property(TARGET ${target_name} APPEND_STRING PROPERTY
LINK_FLAGS " -Wl,-brtl")
endif()
+
+ check_linker_flag(CXX "-Wl,-Bsymbolic-functions"
MaskRay wrote:
Add a comment what ta
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/139891
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/naveen-seth updated
https://github.com/llvm/llvm-project/pull/139457
>From 5f246435b9784113ada3f82328433487391f40ab Mon Sep 17 00:00:00 2001
From: naveen-seth
Date: Sun, 11 May 2025 14:24:00 +
Subject: [PATCH 1/7] [clang] Enforce 1-based indexing for command line source
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp,c,h --
clang/lib/Headers/andes_vector.h
clang/test/Code
wenju-he wrote:
@frasercrmck please help to review, thanks.
https://github.com/llvm/llvm-project/pull/140008
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wenju-he created
https://github.com/llvm/llvm-project/pull/140008
The previous method splits vector data into two halves. shuffle_vector
concatenates the two results into a vector data of original size. This PR
eliminates the use of shuffle_vector.
>From c17e8d57045e32d4602
ChuanqiXu9 wrote:
> @emaxx-google, @ilya-biryukov, do you think we can move forward even though
> don't have a reproducer yet? It feels we are stuck on that one case on a
> downstream client -- worst case maybe the devs can work it around by changing
> source code for that one case?
Yes, I'm
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jim Lin (tclin914)
Changes
This patch implements clang intrinsic support for XAndesVPackFPH.
The document for the intrinsics can be found at:
https://github.com/andestech/andes-vector-intrinsic-doc/blob/ast-v5_4_0-release-v5/auto-generated
llvmbot wrote:
@llvm/pr-subscribers-backend-risc-v
Author: Jim Lin (tclin914)
Changes
This patch implements clang intrinsic support for XAndesVPackFPH.
The document for the intrinsics can be found at:
https://github.com/andestech/andes-vector-intrinsic-doc/blob/ast-v5_4_0-release-v5/auto-
https://github.com/brad0 updated https://github.com/llvm/llvm-project/pull/79539
>From 03859cb1b4eb4132c831fdc8a36909df2981292d Mon Sep 17 00:00:00 2001
From: Brad Smith
Date: Thu, 25 Jan 2024 16:20:47 -0500
Subject: [PATCH] [CMake] Add a linker test for -Bsymbolic-functions to AddLLVM
Add a li
https://github.com/naveen-seth updated
https://github.com/llvm/llvm-project/pull/139457
>From 5f246435b9784113ada3f82328433487391f40ab Mon Sep 17 00:00:00 2001
From: naveen-seth
Date: Sun, 11 May 2025 14:24:00 +
Subject: [PATCH 1/7] [clang] Enforce 1-based indexing for command line source
@@ -189,6 +189,13 @@ def main():
default=config.get("clangformat.style", None),
help="passed to clang-format",
),
+p.add_argument(
owenca wrote:
Please move this to just before `--force` to keep the options sorted. (We can
also move `-
brad0 wrote:
@rorth Thanks. It was more so the native linker I wasn't sure about.
https://github.com/llvm/llvm-project/pull/79539
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca edited
https://github.com/llvm/llvm-project/pull/137609
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca edited
https://github.com/llvm/llvm-project/pull/137609
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -58,8 +58,17 @@ void func(int n) {
// expected-error@+1 {{argument to 'partial' clause must be a strictly
positive integer value}}
#pragma omp unroll partial(0)
for (int i = 0; i < n; ++i) {}
-
- // expected-error@+1 {{directive '#pragma omp unroll' cannot conta
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/137609
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -189,6 +189,13 @@ def main():
default=config.get("clangformat.style", None),
help="passed to clang-format",
),
+p.add_argument(
+"--fallback-style",
+help="The name of the predefined style used as a "
+"fallback in case clang-f
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/138103
>From 65ad7a888dafc9992793baee8a1ccc27f1b79fa5 Mon Sep 17 00:00:00 2001
From: Jannick Kremer
Date: Thu, 1 May 2025 18:37:43 +0900
Subject: [PATCH 1/5] [libclang/python] Add typing annotations for the Cursor
https://github.com/DeinAlptraum edited
https://github.com/llvm/llvm-project/pull/138103
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1552,68 +1553,85 @@ class Cursor(Structure):
_fields_ = [("_kind_id", c_int), ("xdata", c_int), ("data", c_void_p * 3)]
+_tu: TranslationUnit
+
+# This ensures that no operations are possible on null cursors
+# by guarding all method calls with a not-null
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/139457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
LGTM with a few nits. Please give other reviewers some time to chime in
https://github.com/llvm/llvm-project/pull/139457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
@@ -0,0 +1,19 @@
+// Related to #139457
+// Clang uses 1-based indexing for source locations given from the
command-line.
+// Verify that `clang-refactor` rejects 0 as an invalid value for line or
column number.
+
+// For range start:
+// RUN: not clang-refactor local-rename -se
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/139457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,12 @@
+// Related to #139375
MaskRay wrote:
I don't why some regression tests are named `crash-*`, but I would not name
this `crash-*`.
I'd just name this source-loc-zero.cpp and change this comment to `//
Regression test for #139375`
https://githu
@@ -89,8 +92,13 @@ struct ParsedSourceRange {
// probably belongs to the filename which menas the whole
// string should be parsed.
RangeSplit.first = Str;
- } else
+ } else {
+// Column and line numbers are 1-based
Mas
https://github.com/tJener edited
https://github.com/llvm/llvm-project/pull/139990
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tJener updated
https://github.com/llvm/llvm-project/pull/139990
>From c7f23d4280de89810606b5c34d7fb8d03d7a9c3f Mon Sep 17 00:00:00 2001
From: Eric Li
Date: Wed, 14 May 2025 21:21:07 -0400
Subject: [PATCH] [libTooling] Fix `constructExprArgs` for direct-init and
implicit cons
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
Adjust to the new way the ast walker is doing it.
---
Full diff: https://github.com/llvm/llvm-project/pull/14.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/Interp.cpp (+13)
- (modified) c
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/14
Adjust to the new way the ast walker is doing it.
>From 91b2beec47f191dc33714b82363a5b6ebe68eef3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Thu, 15 May 2025 06:13:12 +0200
Subject: [PAT
https://github.com/naveen-seth updated
https://github.com/llvm/llvm-project/pull/139457
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sa
https://github.com/tJener updated
https://github.com/llvm/llvm-project/pull/139990
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-se
https://github.com/naveen-seth updated
https://github.com/llvm/llvm-project/pull/139457
>From 5f246435b9784113ada3f82328433487391f40ab Mon Sep 17 00:00:00 2001
From: naveen-seth
Date: Sun, 11 May 2025 14:24:00 +
Subject: [PATCH 1/6] [clang] Enforce 1-based indexing for command line source
https://github.com/el-ev closed https://github.com/llvm/llvm-project/pull/139508
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
el-ev wrote:
### Merge activity
* **May 14, 10:53 PM EDT**:
[Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/139495)
couldn't merge this PR because it had conflicts with the trunk branch.
https://github.com/llvm/llvm-project/pull/139495
https://github.com/el-ev updated
https://github.com/llvm/llvm-project/pull/139508
>From eb15367085a381955140982177e95ad94a645e92 Mon Sep 17 00:00:00 2001
From: Iris Shi <0...@owo.li>
Date: Mon, 12 May 2025 13:19:06 +0800
Subject: [PATCH 1/3] [RISCV][MC] Add support for Q extension
---
.../Driv
https://github.com/el-ev closed https://github.com/llvm/llvm-project/pull/139369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/el-ev edited https://github.com/llvm/llvm-project/pull/139508
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/el-ev edited https://github.com/llvm/llvm-project/pull/139495
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
@matts1 Congratulations on having your first Pull Request (PR) merged into the
LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a
problem with a build,
Author: Iris Shi
Date: 2025-05-15T10:51:06+08:00
New Revision: 1e503d08e1b6a285608e266acafd40eb2be5ca83
URL:
https://github.com/llvm/llvm-project/commit/1e503d08e1b6a285608e266acafd40eb2be5ca83
DIFF:
https://github.com/llvm/llvm-project/commit/1e503d08e1b6a285608e266acafd40eb2be5ca83.diff
LOG:
https://github.com/ChuanqiXu9 closed
https://github.com/llvm/llvm-project/pull/130712
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Matt
Date: 2025-05-15T10:50:45+08:00
New Revision: 4630d464c5cfd3f2ccdf3ac167694977736ce00e
URL:
https://github.com/llvm/llvm-project/commit/4630d464c5cfd3f2ccdf3ac167694977736ce00e
DIFF:
https://github.com/llvm/llvm-project/commit/4630d464c5cfd3f2ccdf3ac167694977736ce00e.diff
LOG: [cl
https://github.com/ChuanqiXu9 approved this pull request.
https://github.com/llvm/llvm-project/pull/130712
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
el-ev wrote:
### Merge activity
* **May 14, 10:50 PM EDT**: A user started a stack merge that includes this
pull request via
[Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/139369).
https://github.com/llvm/llvm-project/pull/139369
_
https://github.com/el-ev updated
https://github.com/llvm/llvm-project/pull/139369
>From eb15367085a381955140982177e95ad94a645e92 Mon Sep 17 00:00:00 2001
From: Iris Shi <0...@owo.li>
Date: Mon, 12 May 2025 13:19:06 +0800
Subject: [PATCH 1/2] [RISCV][MC] Add support for Q extension
---
.../Driv
Author: Jim Lin
Date: 2025-05-15T10:14:58+08:00
New Revision: 2a8960e48b178fb2533e71bbe164e9f383046114
URL:
https://github.com/llvm/llvm-project/commit/2a8960e48b178fb2533e71bbe164e9f383046114
DIFF:
https://github.com/llvm/llvm-project/commit/2a8960e48b178fb2533e71bbe164e9f383046114.diff
LOG:
mizvekov wrote:
> Thank you for your feedback. I added more tests. By "function templates with
> placeholder return types", do you mean something similar to create_object4 in
> my test case?
Yep. That's what I meant.
https://github.com/llvm/llvm-project/pull/139973
___
https://github.com/albus-droid edited
https://github.com/llvm/llvm-project/pull/139986
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,296 @@
+//===--- OverridePureVirtuals.cpp *-
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
@@ -0,0 +1,296 @@
+//===--- OverridePureVirtuals.cpp *-
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
@@ -0,0 +1,296 @@
+//===--- OverridePureVirtuals.cpp *-
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/marcogmaia commented:
@zwuis, the second round of review was applied.
https://github.com/llvm/llvm-project/pull/139348
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
https://github.com/marcogmaia edited
https://github.com/llvm/llvm-project/pull/139348
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,296 @@
+//===--- OverridePureVirtuals.cpp *-
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/marcogmaia updated
https://github.com/llvm/llvm-project/pull/139348
>From 76503bd8f5618b710e2909d1303de5d34723 Mon Sep 17 00:00:00 2001
From: Marco Maia
Date: Sat, 10 May 2025 00:48:39 -0300
Subject: [PATCH 01/10] [clangd] Add tweak to add pure virtual overrides
---
...
@@ -3221,15 +3223,33 @@ static void sortCppIncludes(const FormatStyle &Style,
stable_sort(Indices, [&](unsigned LHSI, unsigned RHSI) {
const auto LHSFilenameLower = Includes[LHSI].Filename.lower();
const auto RHSFilenameLower = Includes[RHSI].Filename.lower();
-
https://github.com/NexMing edited
https://github.com/llvm/llvm-project/pull/139857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Qinkun Bao (qinkunbao)
Changes
Reverts llvm/llvm-project#138983
---
Full diff: https://github.com/llvm/llvm-project/pull/139987.diff
14 Files Affected:
- (modified) clang-tools-extra/clangd/ModulesBuilder.cpp (+1-2)
- (modi
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
Author: Qinkun Bao (qinkunbao)
Changes
Reverts llvm/llvm-project#138983
---
Full diff: https://github.com/llvm/llvm-project/pull/139987.diff
14 Files Affected:
- (modified) clang-tools-extra/clangd/ModulesBuilder.cpp (+1-2)
- (modified
https://github.com/qinkunbao closed
https://github.com/llvm/llvm-project/pull/139987
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Qinkun Bao
Date: 2025-05-14T21:02:57-04:00
New Revision: 18b885f66babff3a10451bc811ffc077d61ed8ee
URL:
https://github.com/llvm/llvm-project/commit/18b885f66babff3a10451bc811ffc077d61ed8ee
DIFF:
https://github.com/llvm/llvm-project/commit/18b885f66babff3a10451bc811ffc077d61ed8ee.diff
LO
https://github.com/qinkunbao created
https://github.com/llvm/llvm-project/pull/139987
Reverts llvm/llvm-project#138983
>From 1399d288c999ae0fad0b7fd6c030bc336af2c4e8 Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Wed, 14 May 2025 21:02:29 -0400
Subject: [PATCH] Revert "[clang][modules] Timest
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
https://github.com/albus-droid created
https://github.com/llvm/llvm-project/pull/139986
This fixes #139268
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Rahul Joshi (jurahul)
Changes
Adopt simplied `getTrailingObjects` variants that are not templated and/or
return ArrayRef in Decl/DeclTemplate .h/.cpp files.
---
Full diff: https://github.com/llvm/llvm-project/pull/139977.diff
4 Files Af
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/139977
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Rahul Joshi (jurahul)
Changes
Use non-templated form of `getTrailingObjects` when using a single trailing
type in `TrailingObjects`.
---
Full diff: https://github.com/llvm/llvm-project/pull/139974.diff
2 Files Affected:
- (modified) cl
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/139974
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -484,7 +484,7 @@ template
void utf8_to_utf16_in_ok(const std::codecvt& cvt)
{
// UTF-8 string of 1-byte CP, 2-byte CP, 3-byte CP and 4-byte CP
const unsigned char input[] = "b\u0448\u\U0010";
- const char16_t expected[] = {'b', 0x0448, 0x, 0xDBEA, 0xDEAA
alexfh wrote:
@cor3ntin we've root-caused an assertion failure in Clang to this commit. It
reproduces at least up to 1778d3b8245b9a7787bbd0b00f60f879ed4689c9.
The test case is being reduced, but maybe you can spot something obviously
wrong from the assertion failure and the stack trace:
```
as
https://github.com/andykaylor updated
https://github.com/llvm/llvm-project/pull/139768
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,san
rnk wrote:
> Or we think no one else does this anchor business, and in that case... why
> are we doing it?
Anchoring debug information out of a header and into a .cpp file can be a
powerful optimization. I added a key method to Sema in
586f65d31f32ca6bc8cfdb8a4f61bee5057bf6c8 and it resulted
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/139369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/139849
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/139977
Adopt simplied `getTrailingObjects` variants that are not templated and/or
return ArrayRef in Decl/DeclTemplate .h/.cpp files.
Rate limit · GitHub
body {
backgrou
@@ -710,4 +710,17 @@ void AnnotateIgnoreWritesEnd(const char *file, int line);
#define LLVM_PREFERRED_TYPE(T)
#endif
+/// \macro LLVM_VIRTUAL_ANCHOR_FUNCTION
+/// This macro is used to adhere to LLVM's policy that each class with a vtable
+/// must have at least one out-of-lin
https://github.com/rnk commented:
I thought we'd have more anchors, honestly. Maybe these are just the overriding
ones.
https://github.com/llvm/llvm-project/pull/139614
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi
https://github.com/rnk edited https://github.com/llvm/llvm-project/pull/139614
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-aarch64-sve-vls`
running on `linaro-g3-01` while building `clang,llvm` at step 7 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/143/builds/7708
Here is the relevant piece of th
https://github.com/EricWF approved this pull request.
https://github.com/llvm/llvm-project/pull/138708
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mizvekov commented:
Thanks. This needs a few more tests.
* A few tests to make sure we don't produce false positives.
* Tests with function templates with placeholder return types.
We should never be performing copy elision for these, due to implementation
limitations.
htt
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/139965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/grigorypas created
https://github.com/llvm/llvm-project/pull/139973
This PR aims at adding warning similar to the one in GCC (-Wnrvo) that targets
missed opportunities for Named Return Value Optimization (NRVO). The warning is
not enabled by default.
cc: @WenleiHe
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/139974
Use non-templated form of `getTrailingObjects` when using a single trailing
type in `TrailingObjects`.
Rate limit · GitHub
body {
background-color: #f6f8fa;
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Grigory Pastukhov (grigorypas)
Changes
This PR aims at adding warning similar to the one in GCC (-Wnrvo) that targets
missed opportunities for Named Return Value Optimization (NRVO). The warning is
not enabled by default.
cc: @WenleiHe
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
@@ -484,7 +484,7 @@ template
void utf8_to_utf16_in_ok(const std::codecvt& cvt)
{
// UTF-8 string of 1-byte CP, 2-byte CP, 3-byte CP and 4-byte CP
const unsigned char input[] = "b\u0448\u\U0010";
- const char16_t expected[] = {'b', 0x0448, 0x, 0xDBEA, 0xDEAA
@@ -34,6 +34,6 @@ int main(int, char**) {
assert(from_next - from == 9);
assert(to_next - to == 9);
for (unsigned i = 0; i < 9; ++i)
-assert(to[i] == from[i]);
+assert(static_cast(to[i]) == static_cast(from[i]));
cor3ntin wrote:
I removed the red
@@ -618,7 +618,7 @@ template
void utf8_to_utf16_in_error(const std::codecvt&
cvt) {
// UTF-8 string of 1-byte CP, 2-byte CP, 3-byte CP, 4-byte CP
const unsigned char input[] = "b\u0448\uD700\U0010";
- const char16_t expected[] = {'b', 0x0448, 0xD700, 0xDBEA, 0xDEAA
@@ -484,7 +484,7 @@ template
void utf8_to_utf16_in_ok(const std::codecvt& cvt)
{
// UTF-8 string of 1-byte CP, 2-byte CP, 3-byte CP and 4-byte CP
const unsigned char input[] = "b\u0448\u\U0010";
- const char16_t expected[] = {'b', 0x0448, 0x, 0xDBEA, 0xDEAA
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/138708
>From c23bf23ddc8e1c8f50a57fcaf74682e86d8ade16 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Tue, 6 May 2025 17:14:35 +0200
Subject: [PATCH 1/9] [Clang] Add warnings when mixing different charN_t types
c
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions c,h,cpp --
clang/test/CodeGen/ms-secure-hotpatch-bad-file.c
https://github.com/kikairoya closed
https://github.com/llvm/llvm-project/pull/138773
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/EricWF commented:
I don't mean to be difficult, but I'm always weary about removing test
coverage, and I'm not yet convinced that the tests didn't intend to test these
sketchy conversions.
The way it uses the typedefs elsewhere in the same tests, while pinning some of
the t
@@ -123,7 +123,7 @@ _LIBCPP_HIDE_FROM_ABI constexpr void __encode(_OutIt&
__out_it, char32_t __value
_LIBCPP_ASSERT_UNCATEGORIZED(__is_scalar_value(__value), "an invalid unicode
scalar value results in invalid UTF-16");
if (__value < 0x1) {
-*__out_it++ = __value
@@ -123,7 +123,7 @@ _LIBCPP_HIDE_FROM_ABI constexpr void __encode(_OutIt&
__out_it, char32_t __value
_LIBCPP_ASSERT_UNCATEGORIZED(__is_scalar_value(__value), "an invalid unicode
scalar value results in invalid UTF-16");
if (__value < 0x1) {
-*__out_it++ = __value
@@ -484,7 +484,7 @@ template
void utf8_to_utf16_in_ok(const std::codecvt& cvt)
{
// UTF-8 string of 1-byte CP, 2-byte CP, 3-byte CP and 4-byte CP
const unsigned char input[] = "b\u0448\u\U0010";
- const char16_t expected[] = {'b', 0x0448, 0x, 0xDBEA, 0xDEAA
1 - 100 of 455 matches
Mail list logo
