https://github.com/DeinAlptraum approved this pull request.
Thanks for the PR, LGTM
I'll merge this once you've checked the private email setting as described in
the bot comment.
CC @Endilll :
Seems like we're missing a bunch of `TypeKind`s, currently missing 33 - 39 and
179 - 181 enum variant
Himadhith wrote:
@lei137 @amy-kwan @tonykuttai
https://github.com/llvm/llvm-project/pull/142723
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -160,7 +160,7 @@ template struct X;
// Make sure that the instantiated constructor initializes start and
// end properly.
-// CHECK-LABEL: define linkonce_odr void @_ZN1XIiEC2ERKS0_(ptr {{[^,]*}}
%this, ptr noundef nonnull align {{[0-9]+}} dereferenceable({{[0-9]+}}) %othe
@@ -4451,6 +4451,8 @@ defm ptrauth_init_fini_address_discrimination :
OptInCC1FFlag<"ptrauth-init-fini
"Enable address discrimination of function pointers in init/fini arrays">;
defm ptrauth_elf_got : OptInCC1FFlag<"ptrauth-elf-got", "Enable authentication
of pointers from G
https://github.com/ojhunt commented:
This basically looks good to me, sans the "can this just be on by default for
elf/linux/etc" question
https://github.com/llvm/llvm-project/pull/140276
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
@@ -1883,6 +1883,9 @@ void Clang::AddAArch64TargetArgs(const ArgList &Args,
Args.addOptInFlag(
CmdArgs, options::OPT_fptrauth_function_pointer_type_discrimination,
options::OPT_fno_ptrauth_function_pointer_type_discrimination);
+ Args.addOptInFlag(
--
https://github.com/ojhunt closed
https://github.com/llvm/llvm-project/pull/139790
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
brunodf-snps wrote:
Could someone with commit access please merge this? Thanks!
https://github.com/llvm/llvm-project/pull/137719
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/legrosbuffle approved this pull request.
https://github.com/llvm/llvm-project/pull/142147
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ojhunt edited
https://github.com/llvm/llvm-project/pull/140276
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Endilll wrote:
> Seems like we're missing a bunch of `TypeKind`s, currently missing 33 - 39
> and 179 - 181 enum variants. 180 and 181 were only added the other day, we
> might want to look into some way to test these automatically to ensure that
> any additions are also added on Python side i
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/138282
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,40 @@
+//===--- UseEnumClassCheck.h - clang-tidy ---*- 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: Apa
https://github.com/vbvictor approved this pull request.
LGTM, few nits with docs.
Ping, @HerrCai0907, @carlosgalvezp, @PiotrZSL if you wish to leave a review.
I suppose in 1-2 weeks I'll land the PR.
https://github.com/llvm/llvm-project/pull/138282
__
DeinAlptraum wrote:
> Time to put libclang to use, and parse cindex.h?
Probably, though I'll check if there are any more direct options first. I don't
quite like the idea of hardcoding source file paths in the tests of a binding.
https://github.com/llvm/llvm-project/pull/142634
___
Author: Timm Baeder
Date: 2025-06-04T10:07:48+02:00
New Revision: b36e161a09bdebbabe159598778abb011303a9eb
URL:
https://github.com/llvm/llvm-project/commit/b36e161a09bdebbabe159598778abb011303a9eb
DIFF:
https://github.com/llvm/llvm-project/commit/b36e161a09bdebbabe159598778abb011303a9eb.diff
L
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/142120
>From 4669be22f731faf9ce0985aa634166c30c7025c6 Mon Sep 17 00:00:00 2001
From: Jannick Kremer
Date: Fri, 30 May 2025 19:54:15 +0900
Subject: [PATCH] [libclang/python] Derive library function types from
anno
@@ -0,0 +1,40 @@
+//===--- UseEnumClassCheck.h - clang-tidy ---*- 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: Apa
Author: Oliver Hunt
Date: 2025-06-04T00:21:20-07:00
New Revision: 93314bd9462d5c41a23fb402be7ae0c7d099e274
URL:
https://github.com/llvm/llvm-project/commit/93314bd9462d5c41a23fb402be7ae0c7d099e274
DIFF:
https://github.com/llvm/llvm-project/commit/93314bd9462d5c41a23fb402be7ae0c7d099e274.diff
L
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/142720
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/anutosh491 created
https://github.com/llvm/llvm-project/pull/142749
As can be seen through the docs
(https://github.com/llvm/llvm-project/blob/7e1fa09ce2a228c949ce4490c98f2c73ed8ada00/clang/docs/LanguageExtensions.rst#c-keywords-supported-in-all-language-modes),
Clang suppor
vbvictor wrote:
I have a mixed feeling about whether we should always exclude all coroutines by
default or not.
When a coroutine has only one `co-operator` and nothing else should be safe to
use a reference parameter? Am I wrong?
So I think to be "on a safe side" I'd suggest a new option "Ingo
vbvictor wrote:
@stellar-aria, 21th LLVM release will be soon and the PR is 95% ready to land
even if some corner-cases are not matched.
Do you wish to rebase on fresh main and after a quick re-review the check
should be ready to land.
https://github.com/llvm/llvm-project/pull/127430
_
anutosh491 wrote:
On **Main**
```
anutosh491@vv-nuc:/build/anutosh491/llvm-project/build/bin$ ./clang-repl
clang-repl> _Alignas(16) int x;
clang-repl> int align = _Alignof(double);
clang-repl> _Atomic int atomic_var = 0;
clang-repl> _Complex double complex_val = 1.0 + 2.0i;
clang-repl> _Float16 f
anutosh491 wrote:
The error comes from here to be precise
https://github.com/llvm/llvm-project/blob/41841e625db8d14d6701e7cb211b2fcab6a32a50/clang/lib/Parse/Parser.cpp#L1026-L1028
`IsDeclarationStatement` should be true but return false enabling a faulty call
to `ParseTopLevelStmtDecl`
https:/
https://github.com/Keenuts updated
https://github.com/llvm/llvm-project/pull/138530
From 8c405fefdb31200930b9a690df635aff7775f602 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nathan=20Gau=C3=ABr?=
Date: Wed, 30 Apr 2025 11:06:55 +0200
Subject: [PATCH 1/8] [HLSL] Implement vk::ext_builtin_input attr
@@ -140,6 +140,11 @@ def SharedVar : SubsetSubjecthasGlobalStorage() && !S->getTLSKind()}],
"global variables">;
+def HLSLInputBuiltin : SubsetSubjecthasGlobalStorage() &&
+S->getStorageClass()==StorageClass::SC_Static &&
+
anutosh491 wrote:
Should help us in the long run and fix the errors we see while including
input/output based header on xeus-cpp-lite (happens cause these internally
reference `_Noreturn` that clang-repl currently fails to understand)
?
I'm in the process of reviewing old PR to lower the number of open ones for
ease of tracking.
This PR could be reopened if needed.
https://github.com/llvm/llvm-project/pull/123734
___
vbvictor wrote:
@thorsten-klein, gentle ping, do you still wish to work on this check?
21th LLVM release will be soon, If you wish to have your changes in the
upcoming release please rebase on main and fix issues suggested by HerrCai0907.
https://github.com/llvm/llvm-project/pull/124265
___
Author: Morris Hafner
Date: 2025-06-04T11:52:47+02:00
New Revision: 4d6e44db8726d32e0edd47f41baf157986412858
URL:
https://github.com/llvm/llvm-project/commit/4d6e44db8726d32e0edd47f41baf157986412858
DIFF:
https://github.com/llvm/llvm-project/commit/4d6e44db8726d32e0edd47f41baf157986412858.diff
https://github.com/mmha closed https://github.com/llvm/llvm-project/pull/142673
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/davemgreen created
https://github.com/llvm/llvm-project/pull/142760
This updates the element types used in the new __Int8x8_t types added in
#126945, mostly to allow C++ name mangling in ItaniumMangling
mangleAArch64VectorBase to work correctly. Char is replaced by SignedCha
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: David Green (davemgreen)
Changes
This updates the element types used in the new __Int8x8_t types added in #126945, mostly to allow C++ name mangling in ItaniumMangling
mangleAArch64VectorBase to work correctly. Char is replaced by SignedCh
Author: Orlando Cazalet-Hyams
Date: 2025-06-04T10:53:29+01:00
New Revision: ac42923c2defe51fcc9220f68d50b33b5e872933
URL:
https://github.com/llvm/llvm-project/commit/ac42923c2defe51fcc9220f68d50b33b5e872933
DIFF:
https://github.com/llvm/llvm-project/commit/ac42923c2defe51fcc9220f68d50b33b5e8729
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Anutosh Bhat (anutosh491)
Changes
As can be seen through the docs
(https://github.com/llvm/llvm-project/blob/7e1fa09ce2a228c949ce4490c98f2c73ed8ada00/clang/docs/LanguageExtensions.rst#c-keywords-supported-in-all-language-modes),
Clang sup
vbvictor wrote:
Gentle ping @PiotrZSL
https://github.com/llvm/llvm-project/pull/126434
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Michael137 wrote:
> While logically correct I think we might have to teach LLDB how to handle
> this. LLDB has a special "frame recognizer" where it looks for frames using
> this fake debug info mechanism (e.g. __builtin_verbose_trap) and it assumes
> the frame below it is the real source code
vbvictor wrote:
Closed as further review happens in
https://github.com/llvm/llvm-project/pull/131804
https://github.com/llvm/llvm-project/pull/123734
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
https://github.com/DeNiCoN closed
https://github.com/llvm/llvm-project/pull/123734
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Endilll wrote:
> > Time to put libclang to use, and parse cindex.h?
>
> Probably, though I'll check if there are any more direct options first. I
> don't quite like the idea of hardcoding source file paths in bindings tests.
If that helps, it should be available using a relatively stable path,
https://github.com/usx95 edited https://github.com/llvm/llvm-project/pull/142313
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,63 @@
+// RUN: %check_clang_tidy %s performance-noexcept-move-constructor %t -- --
-fexceptions
+
+// RUN: %check_clang_tidy -check-suffix=CONFIG %s
performance-noexcept-move-constructor,performance-noexcept-destructor %t -- \
+// RUN: -config="{CheckOptions:
{perfor
@@ -0,0 +1,63 @@
+// RUN: %check_clang_tidy %s performance-noexcept-move-constructor %t -- --
-fexceptions
vbvictor wrote:
Generally with options-test-file we do not need to check basic functionality
since it should be covered in main test file
https://github
https://github.com/vbvictor commented:
@Nechda, gentle ping, do you still wish to work on this PR? 21th LLVM release
will be soon, If you wish to have your changes in the upcoming release please
rebase on main and fix suggestions.
https://github.com/llvm/llvm-project/pull/126897
__
vbvictor wrote:
@khuldraeseth, gentle ping, do you still wish to work on this check? 21th LLVM
release will be soon, If you wish to have your changes in the upcoming release
please rebase on main and fix suggested issues
https://github.com/llvm/llvm-project/pull/129406
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/126897
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Keenuts wrote:
There is one failure on the CI, in lldb, an unrelated timeout
https://github.com/llvm/llvm-project/pull/138530
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Keenuts closed
https://github.com/llvm/llvm-project/pull/138530
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Keenuts updated
https://github.com/llvm/llvm-project/pull/141759
From 4653aca0444c9ce14893e877eeb88241c9b01934 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nathan=20Gau=C3=ABr?=
Date: Mon, 5 May 2025 18:01:17 +0200
Subject: [PATCH] [HLSL][SPIR-V] Handle SV_Postion builtin in PS
https://github.com/Keenuts edited
https://github.com/llvm/llvm-project/pull/141759
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/xlauko edited
https://github.com/llvm/llvm-project/pull/142690
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/xlauko approved this pull request.
lgtm
https://github.com/llvm/llvm-project/pull/142690
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ArtyomZabroda updated
https://github.com/llvm/llvm-project/pull/142278
>From 6fc280bb5583ee4f1713cb1447b8b86993b7abb7 Mon Sep 17 00:00:00 2001
From: Artyom Zabroda
Date: Sat, 31 May 2025 18:44:21 +0300
Subject: [PATCH 1/2] [clang] Fix bad error recovery when classes are defin
@@ -220,6 +220,14 @@ static ExprResult EvaluateAtomicConstraint(
if (Inst.isInvalid())
return ExprError();
+if (const TemplateTypeParmType *TTPT =
+
dyn_cast(AtomicExpr->getType().getDesugaredType(S.Context)))
{
+ TemplateTypeParmDecl *TTPD = TTPT->
https://github.com/ArtyomZabroda edited
https://github.com/llvm/llvm-project/pull/142278
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ArtyomZabroda edited
https://github.com/llvm/llvm-project/pull/142278
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vortex73 updated
https://github.com/llvm/llvm-project/pull/140112
>From 322c1cfb925f3073f3d3b30abe1b2e35ae7745f3 Mon Sep 17 00:00:00 2001
From: Narayan Sreekumar
Date: Thu, 15 May 2025 23:13:50 +0530
Subject: [PATCH 1/6] [LLVM ABI] The Typesystem
---
llvm/include/llvm/ABI/T
rorth wrote:
> and couldn't find much on it either, but that sounds like
>
> > Alternatively, as I suggested, one could wrap the actual python -m unittest
> > discover invocation with a check if libclang.so is loadable at all, only
> > then running the actual test.
Indeed: one should move the
rorth wrote:
> @rorth Now that #142371 is merged, can you re-evaluate this PR and update the
> description?
This PR changed the output from completely silent to excessively verbose: I now
get
```
FAILED: tools/clang/bindings/python/tests/CMakeFiles/check-clang-python
/var/llvm/local-i386-rele
@@ -136,9 +136,69 @@ const llvm::abi::Type
*QualTypeMapper::convertRecordType(const RecordType *RT) {
if (RD->isUnion())
return convertUnionType(RD);
+
+ // Handle C++ classes with base classes
+ auto *const CXXRd = dyn_cast(RD);
+ if (CXXRd && (CXXRd->getNumBases()
@@ -136,9 +136,69 @@ const llvm::abi::Type
*QualTypeMapper::convertRecordType(const RecordType *RT) {
if (RD->isUnion())
return convertUnionType(RD);
+
+ // Handle C++ classes with base classes
+ auto *const CXXRd = dyn_cast(RD);
+ if (CXXRd && (CXXRd->getNumBases()
naveen-seth wrote:
Ping
https://github.com/llvm/llvm-project/pull/141695
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -136,9 +136,69 @@ const llvm::abi::Type
*QualTypeMapper::convertRecordType(const RecordType *RT) {
if (RD->isUnion())
return convertUnionType(RD);
+
+ // Handle C++ classes with base classes
+ auto *const CXXRd = dyn_cast(RD);
+ if (CXXRd && (CXXRd->getNumBases()
LU-JOHN wrote:
@dtcxzyw can you please review? Thanks.
https://github.com/llvm/llvm-project/pull/135079
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dmpolukhin wrote:
@HerrCai0907 @carlosgalvezp @PiotrZSL one more friendly ping. Please take a
look, the change is benign and trivial one liner + test. I think it will really
improve this check and avoid long hours for developers to figure out dangling
references.
https://github.com/llvm/llvm-
https://github.com/ganenkokb-yandex updated
https://github.com/llvm/llvm-project/pull/138838
>From 0a5beb71cb46ad5ef2df753098a6742ffeef3a71 Mon Sep 17 00:00:00 2001
From: Evianaive <153540...@qq.com>
Date: Tue, 25 Mar 2025 01:54:06 +0800
Subject: [PATCH 1/5] Implement missing visit function
---
@@ -136,9 +136,69 @@ const llvm::abi::Type
*QualTypeMapper::convertRecordType(const RecordType *RT) {
if (RD->isUnion())
return convertUnionType(RD);
+
+ // Handle C++ classes with base classes
+ auto *const CXXRd = dyn_cast(RD);
+ if (CXXRd && (CXXRd->getNumBases()
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/142690
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,35 @@
+.. title:: clang-tidy - cppcoreguidelines-use-enum-class
+
+cppcoreguidelines-use-enum-class
+
+
+Finds unscoped (non-class) ``enum`` declarations and suggests using
+``enum class`` instead.
+
+This check implements `Enum.3
---
https://github.com/flovent updated
https://github.com/llvm/llvm-project/pull/141345
>From e020a00fcaa8283edf5fb1850653d0156918ca85 Mon Sep 17 00:00:00 2001
From: flovent
Date: Sat, 24 May 2025 21:29:53 +0800
Subject: [PATCH 1/2] [clang-tidy] Add check for assignment or comparision
operators' o
@@ -0,0 +1,35 @@
+.. title:: clang-tidy - cppcoreguidelines-use-enum-class
+
+cppcoreguidelines-use-enum-class
+
+
+Finds unscoped (non-class) ``enum`` declarations and suggests using
+``enum class`` instead.
+
+This check implements `Enum.3
---
https://github.com/AaronBallman approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/141695
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/138282
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -8253,6 +8275,19 @@ inline bool Type::isObjectPointerType() const {
return false;
}
+inline bool Type::isCFIUncheckedCalleeFunctionType() const {
+ if (const auto *Fn = getAs())
+return Fn->hasCFIUncheckedCallee();
+ return false;
+}
+
+inline bool Type::isPointer
@@ -0,0 +1,35 @@
+.. title:: clang-tidy - cppcoreguidelines-use-enum-class
+
+cppcoreguidelines-use-enum-class
+
+
+Finds unscoped (non-class) ``enum`` declarations and suggests using
+``enum class`` instead.
+
+This check implements `Enum.3
---
@@ -12711,6 +12711,11 @@ def warn_noderef_on_non_pointer_or_array : Warning<
def warn_noderef_to_dereferenceable_pointer : Warning<
"casting to dereferenceable pointer removes 'noderef' attribute">,
InGroup;
+def warn_cast_discards_cfi_unchecked_callee
+: Warning<"impli
https://github.com/atrosinenko updated
https://github.com/llvm/llvm-project/pull/141573
>From c5a7fea9925c9d4aebf948c9e3d7f0af47893fb7 Mon Sep 17 00:00:00 2001
From: Anatoly Trosinenko
Date: Mon, 26 May 2025 22:28:55 +0300
Subject: [PATCH 1/2] [AArch64][FMV] Enable PAuth and BTI hardening of re
https://github.com/AaronBallman approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/135836
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,35 @@
+.. title:: clang-tidy - cppcoreguidelines-use-enum-class
+
+cppcoreguidelines-use-enum-class
+
+
+Finds unscoped (non-class) ``enum`` declarations and suggests using
+``enum class`` instead.
+
+This check implements `Enum.3
---
https://github.com/frasercrmck updated
https://github.com/llvm/llvm-project/pull/139759
>From c81d3c18784d66f8a40bacb1195c1205c895d806 Mon Sep 17 00:00:00 2001
From: Fraser Cormack
Date: Thu, 27 Feb 2025 15:02:32 +
Subject: [PATCH 1/3] [libclc] Add (fast) normalize to CLC; add half overload
frasercrmck wrote:
> > The OpenCL-CTS doesn't actually test the geometrics builtins with that
> > option - that's just for bruteforce.
>
> Sounds like a bug
>
> > It passes on an OpenCL implementation which advertises support for
> > single-precision denormals, yes.
>
> Ideally would check i
https://github.com/frasercrmck edited
https://github.com/llvm/llvm-project/pull/139759
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1432,14 +1432,26 @@ void
NumericLiteralParser::ParseNumberStartingWithZero(SourceLocation TokLoc) {
Diags.Report(TokLoc, DiagId);
++s;
DigitsBegin = s;
-SawOctalPrefix = true;
+radix = 8;
+s = SkipOctalDigits(s);
+if (s == ThisTokEnd) {
+ /
@@ -1432,14 +1432,26 @@ void
NumericLiteralParser::ParseNumberStartingWithZero(SourceLocation TokLoc) {
Diags.Report(TokLoc, DiagId);
++s;
DigitsBegin = s;
-SawOctalPrefix = true;
+radix = 8;
+s = SkipOctalDigits(s);
+if (s == ThisTokEnd) {
+ /
https://github.com/jmorse approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/134652
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4652,6 +4659,7 @@ llvm::Constant
*CodeGenModule::GetOrCreateMultiVersionResolver(GlobalDecl GD) {
"", Resolver, &getModule());
GIF->setName(ResolverName);
SetCommonAttributes(FD, GIF);
+SetResolverAttrs(cast(*Resolver));
--
marcogmaia wrote:
Ping
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
https://github.com/mmha created https://github.com/llvm/llvm-project/pull/142779
This changes clang::CIRGen::RValue to look like current clang::CodeGen::RValue
which was changed in
84780a/[#86923](https://github.com/llvm/llvm-project/issues/86923). This should
be NFC and is preliminary work fo
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Morris Hafner (mmha)
Changes
This changes clang::CIRGen::RValue to look like current clang::CodeGen::RValue
which was changed in 84780a/[#86923](https://github.com/llvm/llvm-project/issues/86923). This should be
NFC and is preliminary wor
llvmbot wrote:
@llvm/pr-subscribers-clangir
Author: Morris Hafner (mmha)
Changes
This changes clang::CIRGen::RValue to look like current clang::CodeGen::RValue
which was changed in 84780a/[#86923](https://github.com/llvm/llvm-project/issues/86923). This should be
NFC and is preliminary w
kazutakahirata wrote:
Friendly ping. Thanks!
https://github.com/llvm/llvm-project/pull/142406
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kazutakahirata wrote:
Friendly ping. Thanks!
https://github.com/llvm/llvm-project/pull/142407
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rjmccall commented:
LGTM, but I've fixed the release note: this is about the Objective-C exceptions
ABI, not a general `type_info` thing for Objective-C++.
https://github.com/llvm/llvm-project/pull/135562
___
cfe-commits mailing lis
https://github.com/rjmccall edited
https://github.com/llvm/llvm-project/pull/135562
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -317,6 +317,7 @@ Non-comprehensive list of changes in this release
``sizeof`` or ``typeof`` expression. (#GH138444)
- Deprecation warning is emitted for the deprecated
``__reference_binds_to_temporary`` intrinsic.
``__reference_constructs_from_temporary`` should be used
https://github.com/rjmccall edited
https://github.com/llvm/llvm-project/pull/135562
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
eZWALT wrote:
I originally kept the `NumGeneratedLoops` information consistent despite being
partially subsumed by NumGeneratedLoopNests (Note that its not actually the
same, it returns the number of generated loops in total adding nested loops,
but due to the current usage of this semantic i
https://github.com/tgymnich approved this pull request.
https://github.com/llvm/llvm-project/pull/142407
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tgymnich approved this pull request.
https://github.com/llvm/llvm-project/pull/142406
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 359 matches
Mail list logo
