https://github.com/glebov-andrey edited
https://github.com/llvm/llvm-project/pull/114075
___
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: None (Himadhith)
Changes
Support the following packed BCD builtins for PowerPC.
```
__builtin_national2packed - Conversion of National format to Packed decimal
format.
__builtin_packed2national - Conversion of Packed decimal format to n
llvmbot wrote:
@llvm/pr-subscribers-backend-powerpc
@llvm/pr-subscribers-clang-codegen
Author: None (Himadhith)
Changes
Support the following packed BCD builtins for PowerPC.
```
__builtin_national2packed - Conversion of National format to Packed decimal
format.
__builtin_packed2nation
DeinAlptraum wrote:
I am unfortunately not at all familiar with how exactly the other tests are
integrated and produce the outputs you described:
> One problem with all these approaches, as I'd already mentioned, is that the
> test outcome isn't properly reported, nor can one use the usual mach
@@ -129,11 +129,19 @@ std::optional tryExpandAsInteger(StringRef Macro,
const Preprocessor &PP) {
// Parse an integer at the end of the macro definition.
const Token &T = FilteredTokens.back();
- // FIXME: EOF macro token coming from a PCH file on macOS while marked as
-
@@ -0,0 +1,39 @@
+// RUN: %clang_cc1 -triple x86_64-apple-macosx10.15.0 -emit-pch -o %t %s
+// RUN: %clang_analyze_cc1 -triple x86_64-apple-macosx10.15.0 -include-pch %t \
+// RUN: -analyzer-checker=core,apiModeling,unix.StdCLibraryFunctions -verify
%s
+//
+// RUN: %clang_cc1 -
https://github.com/steakhal requested changes to this pull request.
Thank you for the patch.
https://github.com/llvm/llvm-project/pull/142722
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/142722
___
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-static-analyzer-1
Author: Ziqing Luo (ziqingluo-90)
Changes
The function `tryExpandAsInteger` attempts to extract an integer from a macro
definition. Previously, the attempt would fail when the macro is from a PCH,
because the function tried to
https://github.com/ziqingluo-90 created
https://github.com/llvm/llvm-project/pull/142722
The function `tryExpandAsInteger` attempts to extract an integer from a macro
definition. Previously, the attempt would fail when the macro is from a PCH,
because the function tried to access the text buf
github-actions[bot] wrote:
⚠️ We detected that you are using a GitHub private e-mail address to contribute
to the repo. Please turn off [Keep my email addresses
private](https://github.com/settings/emails) setting in your account. See
[LLVM
Discourse](https://discourse.llvm.org/t/hidden-email
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/142720.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/Context.cpp (+12)
- (modified) clang/lib/AST/ByteCode/Context.h (+2)
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/142720
None
>From 22f7f0a3b1a5b740377ce56ea0532b6d4a866aa0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 4 Jun 2025 07:05:29 +0200
Subject: [PATCH] [clang][bytecode][NFC] Cache more integer
tbaederr wrote:
Could you run this through the compile time tracker?
https://github.com/llvm/llvm-project/pull/142713
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3074,6 +3074,39 @@ following way:
Query for this feature with ``__has_builtin(__builtin_offsetof)``.
+``__builtin_get_vtable_pointer``
+
+
+``__builtin_get_vtable_pointer`` loads and authenticates the primary vtable
+pointer from an instanc
https://github.com/ojhunt updated
https://github.com/llvm/llvm-project/pull/139790
>From a933679801bde89b53584c4c65118658477db4da Mon Sep 17 00:00:00 2001
From: Oliver Hunt
Date: Tue, 13 May 2025 13:35:55 -0700
Subject: [PATCH 1/5] [clang][PAC] Add __builtin_get_vtable_pointer
With pointer aut
@@ -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
https://github.com/carlosgalvezp approved this pull request.
LGTM, if all tests pass then this must be dead code! Thanks for cleaning up.
https://github.com/llvm/llvm-project/pull/142654
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://l
@@ -1825,6 +1825,37 @@ static ExprResult PointerAuthStringDiscriminator(Sema
&S, CallExpr *Call) {
return Call;
}
+static ExprResult GetVTablePointer(Sema &S, CallExpr *Call) {
+ if (S.checkArgCount(Call, 1))
+return ExprError();
+ ExprResult ThisArg = S.DefaultFuncti
@@ -1343,12 +1343,21 @@ static bool EvaluateBooleanTypeTrait(Sema &S, TypeTrait
Kind,
if (RD && RD->isAbstract())
return false;
+// LWG3819: For reference_meows_from_temporary traits, && is not added to
shafik wrote:
I apologize, maybe I am jus
https://github.com/Un1q32 updated
https://github.com/llvm/llvm-project/pull/141401
>From 3cf20444591e70a8a6d8782048ca4f6c0e22 Mon Sep 17 00:00:00 2001
From: Un1q32
Date: Mon, 27 Jan 2025 18:00:34 -0500
Subject: [PATCH 1/3] [Clang] Link libgcc_s.1.dylib when building for macOS
10.5 and olde
@@ -1645,14 +1645,15 @@ void DarwinClang::AddLinkRuntimeLibArgs(const ArgList
&Args,
CmdArgs.push_back("-lSystem");
// Select the dynamic runtime library and the target specific static library.
- if (isTargetIOSBased()) {
-// If we are compiling as iOS / simulator,
https://github.com/Un1q32 updated
https://github.com/llvm/llvm-project/pull/141401
>From 3cf20444591e70a8a6d8782048ca4f6c0e22 Mon Sep 17 00:00:00 2001
From: Un1q32
Date: Mon, 27 Jan 2025 18:00:34 -0500
Subject: [PATCH 1/2] [Clang] Link libgcc_s.1.dylib when building for macOS
10.5 and olde
@@ -1645,14 +1645,15 @@ void DarwinClang::AddLinkRuntimeLibArgs(const ArgList
&Args,
CmdArgs.push_back("-lSystem");
// Select the dynamic runtime library and the target specific static library.
- if (isTargetIOSBased()) {
-// If we are compiling as iOS / simulator,
@@ -1645,14 +1645,15 @@ void DarwinClang::AddLinkRuntimeLibArgs(const ArgList
&Args,
CmdArgs.push_back("-lSystem");
// Select the dynamic runtime library and the target specific static library.
- if (isTargetIOSBased()) {
-// If we are compiling as iOS / simulator,
https://github.com/aaupov closed
https://github.com/llvm/llvm-project/pull/142698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/aaupov edited
https://github.com/llvm/llvm-project/pull/142698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/carlocab commented:
Seems reasonable to me, but someone from Apple should probably be the one to ✅
this.
https://github.com/llvm/llvm-project/pull/141401
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org
qinkunbao wrote:
I split this PR into twos
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wangpc-pp approved this pull request.
https://github.com/llvm/llvm-project/pull/142517
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/qinkunbao edited
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Eli Friedman
Date: 2025-06-03T16:22:30-07:00
New Revision: 12f8bf34c3e56f30bda11e0edd92b4ac5914ec47
URL:
https://github.com/llvm/llvm-project/commit/12f8bf34c3e56f30bda11e0edd92b4ac5914ec47
DIFF:
https://github.com/llvm/llvm-project/commit/12f8bf34c3e56f30bda11e0edd92b4ac5914ec47.diff
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Sebastian Pop (sebpop)
Changes
This adds the flag -floop-fuse to the clang and flang drivers.
---
Full diff: https://github.com/llvm/llvm-project/pull/142686.diff
15 Files Affected:
- (modified) clang/include/clang/Basic/CodeGenOptions
https://github.com/rnk approved this pull request.
Yep, feel free to reland this.
https://github.com/llvm/llvm-project/pull/140910
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,23 @@
+// RUN: %clang_cc1 -triple x86_64-unknown-linux-gnu -emit-cir %s -o %t.cir
Andres-Salamanca wrote:
If I add a big-endian target triple, it might cause errors for others who
haven't built LLVM with support for that architecture. Is there a way w
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `flang-aarch64-latest-gcc`
running on `linaro-flang-aarch64-latest-gcc` while building `clang,llvm` at
step 5 "build-unified-tree".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/130/builds/13551
Her
@@ -813,19 +813,19 @@ changes to one object won't affect the others, the
object's initializer will run
once per copy, etc.
Specifically, this warning fires when it detects an object which:
- 1. Is defined as ``inline`` in a header file (so it might get compiled into
multipl
ojhunt wrote:
The docs failure is #142387
https://github.com/llvm/llvm-project/pull/141148
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Oliver Hunt
Date: 2025-06-03T17:57:01-07:00
New Revision: f72054a0ccecb88c694713c44f3f42352d2340a2
URL:
https://github.com/llvm/llvm-project/commit/f72054a0ccecb88c694713c44f3f42352d2340a2
DIFF:
https://github.com/llvm/llvm-project/commit/f72054a0ccecb88c694713c44f3f42352d2340a2.diff
L
ojhunt wrote:
this is tripping on CI and my machine.
To repro I need to delete `$BUILD_DIR/tools/clang/docs` and build with `TZ=UTC
ninja docs-clang-man`
I find it necessary to delete the doc directory to trigger this.
https://github.com/llvm/llvm-project/pull/142387
___
Author: Qinkun Bao
Date: 2025-06-03T20:24:42-04:00
New Revision: e129c3c0117ee8829d9979140a01957b3ec873eb
URL:
https://github.com/llvm/llvm-project/commit/e129c3c0117ee8829d9979140a01957b3ec873eb
DIFF:
https://github.com/llvm/llvm-project/commit/e129c3c0117ee8829d9979140a01957b3ec873eb.diff
LO
https://github.com/qinkunbao closed
https://github.com/llvm/llvm-project/pull/142700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/qinkunbao closed
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shafik approved this pull request.
https://github.com/llvm/llvm-project/pull/141148
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Qinkun Bao
Date: 2025-06-03T20:28:07-04:00
New Revision: 63861d69f0b59787cd8a288543b1434414a2b250
URL:
https://github.com/llvm/llvm-project/commit/63861d69f0b59787cd8a288543b1434414a2b250
DIFF:
https://github.com/llvm/llvm-project/commit/63861d69f0b59787cd8a288543b1434414a2b250.diff
LO
https://github.com/qinkunbao edited
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/142659
>From c3e6f0f64be652259065c4c57b9dec7cc30502d0 Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Tue, 3 Jun 2025 19:29:54 +
Subject: [PATCH 1/4] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
ojhunt wrote:
I can't reproduce this locally so I've ensured my tree is up to date and
pushing again to see if it fails again
https://github.com/llvm/llvm-project/pull/141148
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.o
https://github.com/ojhunt updated
https://github.com/llvm/llvm-project/pull/141148
>From 0db205bb8f6925dd725885395f58a688000c7b8b Mon Sep 17 00:00:00 2001
From: Oliver Hunt
Date: Thu, 22 May 2025 02:38:26 -0700
Subject: [PATCH 1/5] [clang] Correct FixIt ranges for unused capture warnings
Fixes
https://github.com/qinkunbao edited
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -813,19 +813,19 @@ changes to one object won't affect the others, the
object's initializer will run
once per copy, etc.
Specifically, this warning fires when it detects an object which:
- 1. Is defined as ``inline`` in a header file (so it might get compiled into
multipl
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka reopened
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/142700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -813,19 +813,19 @@ changes to one object won't affect the others, the
object's initializer will run
once per copy, etc.
Specifically, this warning fires when it detects an object which:
- 1. Is defined as ``inline`` in a header file (so it might get compiled into
multipl
vitalybuka wrote:
Closed?
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -813,19 +813,19 @@ changes to one object won't affect the others, the
object's initializer will run
once per copy, etc.
Specifically, this warning fires when it detects an object which:
- 1. Is defined as ``inline`` in a header file (so it might get compiled into
multipl
@@ -843,8 +843,8 @@ static const ARMVectorIntrinsicInfo ARMSIMDIntrinsicMap []
= {
NEONMAP1(vrndaq_v, arm_neon_vrinta, Add1ArgType),
NEONMAP0(vrndi_v),
NEONMAP0(vrndiq_v),
- NEONMAP1(vrndm_v, arm_neon_vrintm, Add1ArgType),
- NEONMAP1(vrndmq_v, arm_neon_vrintm, Add1ArgT
https://github.com/qinkunbao closed
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/142659
>From c3e6f0f64be652259065c4c57b9dec7cc30502d0 Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Tue, 3 Jun 2025 19:29:54 +
Subject: [PATCH 1/4] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Qinkun Bao (qinkunbao)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/142700.diff
1 Files Affected:
- (modified) clang/include/clang/Basic/DiagnosticGroups.td (+8-8)
``diff
diff --git a/clang/include/clang/
https://github.com/qinkunbao created
https://github.com/llvm/llvm-project/pull/142700
None
>From e234d999848d00cafa7878f988110e61663eb645 Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Wed, 4 Jun 2025 00:08:04 +
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=
@@ -2595,6 +2595,14 @@ GetX86_64ByValArgumentPair(llvm::Type *Lo, llvm::Type
*Hi,
ABIArgInfo X86_64ABIInfo::
classifyReturnType(QualType RetTy) const {
+ // return int128 as i128
+ if (const BuiltinType *BT = RetTy->getAs()) {
+BuiltinType::Kind k = BT->getKind();
+
@@ -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
@@ -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
https://github.com/PiJoules updated
https://github.com/llvm/llvm-project/pull/135836
>From 21ada1eba526269debf06652b2a8efbf20161f14 Mon Sep 17 00:00:00 2001
From: Leonard Chan
Date: Wed, 9 Apr 2025 14:21:00 -0700
Subject: [PATCH] [clang] Function type attribute to prevent CFI
instrumentation
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/142659
>From c3e6f0f64be652259065c4c57b9dec7cc30502d0 Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Tue, 3 Jun 2025 19:29:54 +
Subject: [PATCH 1/4] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
https://github.com/sarnex ready_for_review
https://github.com/llvm/llvm-project/pull/140910
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
V-FEXrt wrote:
> Can you reshare the original crash you saw, not the minimized godbolt
> example?
https://godbolt.org/z/f4EEWrvac
> Should we change it there as well?
Yeah I think so
https://github.com/llvm/llvm-project/pull/142679
___
cfe-commits
HazardyKnusperkeks wrote:
I'd say yes.
https://github.com/llvm/llvm-project/pull/142104
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ilovepi wrote:
> Right now, since I just ripped the Mustache generator code, it creates
> folders for each namespace and emits the nested entities there. This actually
> represents a problem for template specializations because the code tries to
> write another JSON object (the specialization
@@ -223,21 +272,114 @@ void CIRRecordLowering::fillOutputFields() {
fieldTypes.size() - 1;
// A field without storage must be a bitfield.
assert(!cir::MissingFeatures::bitfields());
+ if (!member.data)
+setBitFieldInfo(member.fieldDecl, memb
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/lib/CodeGen/BackendUtil.cpp clang/lib/Drive
https://github.com/sebpop updated
https://github.com/llvm/llvm-project/pull/142686
>From ff88172ea86363bd25f73fc78af1b8fca2cc4f3d Mon Sep 17 00:00:00 2001
From: Sebastian Pop
Date: Thu, 22 May 2025 13:50:38 +
Subject: [PATCH] add -floop-fuse to clang and flang
---
clang/include/clang/Basi
sarnex wrote:
Dependent commit reapplied.
https://github.com/llvm/llvm-project/pull/140910
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ilovepi wrote:
> I'll investigate this. Although right off the bat, the names in the Infos
> aren't mangled. `Name` and `FullName` are the same for both records, which is
> unfortunate.
OK, well, let's give it a try, and if its too hard lets ... IDK go w/ the YAML
thing, so its correct? Test
@@ -198,37 +198,107 @@ static json::Value extractValue(const TypedefInfo &I) {
}
static json::Value extractValue(const CommentInfo &I) {
- assert((I.Kind == "BlockCommandComment" || I.Kind == "FullComment" ||
- I.Kind == "ParagraphComment" || I.Kind == "TextComment")
delcypher wrote:
@Michael137 Ping
https://github.com/llvm/llvm-project/pull/141997
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -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
efriedma-quic wrote:
If the goal here is gcc compatibility, I'd suggest not hooking up the flag to
anything; the existing LoopFusePass isn't used by anything and hasn't been
touched in years, so it's very likely to have issues.
https://github.com/llvm/llvm-project/pull/142686
_
https://github.com/andykaylor created
https://github.com/llvm/llvm-project/pull/142690
When cir::RecordType was initially upstreamed, we decided that there was no
reason to distinguish between RecordKind::Class and RecordKind::Struct since
they are semantically equivalent. I think this was a m
https://github.com/efriedma-quic closed
https://github.com/llvm/llvm-project/pull/141887
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -198,37 +198,107 @@ static json::Value extractValue(const TypedefInfo &I) {
}
static json::Value extractValue(const CommentInfo &I) {
- assert((I.Kind == "BlockCommandComment" || I.Kind == "FullComment" ||
- I.Kind == "ParagraphComment" || I.Kind == "TextComment")
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Andy Kaylor (andykaylor)
Changes
When cir::RecordType was initially upstreamed, we decided that there was no
reason to distinguish between RecordKind::Class and RecordKind::Struct since
they are semantically equivalent. I think this was a
@@ -198,37 +198,107 @@ static json::Value extractValue(const TypedefInfo &I) {
}
static json::Value extractValue(const CommentInfo &I) {
- assert((I.Kind == "BlockCommandComment" || I.Kind == "FullComment" ||
- I.Kind == "ParagraphComment" || I.Kind == "TextComment")
shafik wrote:
> Based on #141114 and #141858 should this PR be reverted?
A fix was landed: https://github.com/llvm/llvm-project/pull/142498
https://github.com/llvm/llvm-project/pull/95474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
https://github.com/Andres-Salamanca updated
https://github.com/llvm/llvm-project/pull/142041
>From 8fd6f99f59d1bbde98696559141bcfe15920d175 Mon Sep 17 00:00:00 2001
From: Andres Salamanca
Date: Thu, 29 May 2025 16:17:09 -0500
Subject: [PATCH 1/4] Add initial support for bitfields in structs and
@@ -670,12 +671,17 @@ static std::unique_ptr genHTML(const
CommentInfo &I) {
return nullptr;
return std::move(BlockComment);
}
- if (I.Kind == "TextComment") {
-if (I.Text == "")
+
+ case CommentKind::CK_TextComment: {
+if (I.Text.empty())
return n
@@ -45,6 +45,25 @@ enum class InfoType {
IT_typedef
};
+enum class CommentKind {
+ CK_FullComment,
+ CK_ParagraphComment,
+ CK_TextComment,
+ CK_InlineCommandComment,
+ CK_HTMLStartTagComment,
+ CK_HTMLEndTagComment,
+ CK_BlockCommandComment,
+ CK_ParamCommandComment
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Sebastian Pop (sebpop)
Changes
This adds the flag -floop-fuse to the clang and flang drivers.
---
Full diff: https://github.com/llvm/llvm-project/pull/142686.diff
15 Files Affected:
- (modified) clang/include/clang/Basic/CodeGen
https://github.com/sebpop created
https://github.com/llvm/llvm-project/pull/142686
This adds the flag -floop-fuse to the clang and flang drivers.
>From d31011703460db137e050906de501a29c4c24fd5 Mon Sep 17 00:00:00 2001
From: Sebastian Pop
Date: Thu, 22 May 2025 13:50:38 +
Subject: [PATCH]
vsapsai wrote:
> Thanks a lot, I am sorry for misreading or misunderstanding your comments
> too. I hope we can put this behind ourselves.
Appreciate the understanding.
> [skip the technical description]
Thanks for the explanation, it gives a lot of food for thought. Still building
a mental
https://github.com/evelez7 approved this pull request.
https://github.com/llvm/llvm-project/pull/142665
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
evelez7 wrote:
> This is a good start, but I thin we'll want some more tests, and probably
> some unit test coverage. Unittesting is especially nice, since I believe this
> backend doesn't need any of the asset files, right?
Yeah, this doesn't need assets, so I'll be able to call things direct
spall wrote:
I don't fully understand why this was a problem? Can you reshare the original
crash you saw, not the minimized godbolt example?
This for example uses getVectorType
https://github.com/llvm/llvm-project/blob/97686f2cd074a143022397b94ab1af1784ed75f2/clang/lib/Sema/SemaHLSL.cpp#L2170
S
@@ -198,37 +198,107 @@ static json::Value extractValue(const TypedefInfo &I) {
}
static json::Value extractValue(const CommentInfo &I) {
- assert((I.Kind == "BlockCommandComment" || I.Kind == "FullComment" ||
- I.Kind == "ParagraphComment" || I.Kind == "TextComment")
@@ -65,6 +65,35 @@ template <> struct ScalarEnumerationTraits {
}
};
+template <>
+struct llvm::yaml::ScalarEnumerationTraits {
evelez7 wrote:
This doesn't need the `llvm::yaml` qualification.
https://github.com/llvm/llvm-project/pull/142273
__
https://github.com/qinkunbao edited
https://github.com/llvm/llvm-project/pull/142659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mmha approved this pull request.
LGTM (and thanks!)
https://github.com/llvm/llvm-project/pull/142591
___
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 `llvm-x86_64-debian-dylib`
running on `gribozavr4` while building `clang` at step 6
"test-build-unified-tree-check-clang".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/60/builds/29238
Here is the r
1 - 100 of 439 matches
Mail list logo