https://github.com/usx95 updated
https://github.com/llvm/llvm-project/pull/106277
>From b2bb29ec61f4e9a7b3b7f9bcd0f5b7a12c844d14 Mon Sep 17 00:00:00 2001
From: Utkarsh Saxena
Date: Tue, 27 Aug 2024 19:44:34 +
Subject: [PATCH 1/4] [clang] Properly set file and line info for -ftime-trace
---
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 35dfe8013c2e100ffa159ca183430422a370d3a4
fd9f5218e72d0d9b4d4b248383179f6dea01a5c6 --e
@@ -0,0 +1,225 @@
+//===- AMDGPUMCResourceInfo.cpp --- MC Resource Info
--===//
+//
+// 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/cor3ntin approved this pull request.
Now that we are not intending to backport that, LGTM
https://github.com/llvm/llvm-project/pull/106335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
@@ -114,27 +132,46 @@ struct llvm::TimeTraceProfiler {
llvm::get_thread_name(ThreadName);
}
- TimeTraceProfilerEntry *begin(std::string Name,
-llvm::function_ref Detail,
-bool AsyncEvent = false) {
-Sta
@@ -18046,6 +18048,19 @@ void Sema::MarkFunctionReferenced(SourceLocation Loc,
FunctionDecl *Func,
std::make_pair(Func, PointOfInstantiation));
// Notify the consumer that a function was implicitly instantiated.
Consumer.HandleCXXImplici
@@ -18046,6 +18048,19 @@ void Sema::MarkFunctionReferenced(SourceLocation Loc,
FunctionDecl *Func,
std::make_pair(Func, PointOfInstantiation));
// Notify the consumer that a function was implicitly instantiated.
Consumer.HandleCXXImplici
@@ -104,6 +105,23 @@ struct llvm::TimeTraceProfilerEntry {
}
};
+struct InProgressEntry {
+ std::unique_ptr Event;
+ std::vector InstantEvents;
usx95 wrote:
Can you add documentation about InstantEvents and that they are associated with
a parent duration
@@ -152,6 +154,11 @@ timeTraceProfilerBegin(StringRef Name,
TimeTraceProfilerEntry *timeTraceAsyncProfilerBegin(StringRef Name,
StringRef Detail);
+// Mark an instant event.
+void timeTraceProfilerInsert(StringRef Name,
+
yxsamliu wrote:
> @yxsamliu Do you know what the next steps for merging this would be? I'd like
> to get it into the Clang 20 release if possible. The only thing this loses
> currently is managed variables being registered in RDC mode, but I'm going to
> assume that's hardly seen in practice s
@@ -0,0 +1,209 @@
+// RUN: %clang_cc1 -finclude-default-header -x hlsl -triple \
+// RUN: dxil-pc-shadermodel6.3-library %s -fnative-half-type \
+// RUN: -emit-llvm -disable-llvm-passes -o - | FileCheck %s \
+// RUN: --check-prefixes=CHECK,DXIL_CHECK,DXIL_NATIVE_HALF,NATIVE_
https://github.com/ivanaivanovska ready_for_review
https://github.com/llvm/llvm-project/pull/103039
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-llvm-support
@llvm/pr-subscribers-clang
Author: None (ivanaivanovska)
Changes
Added an instant event type and issuing such events each time a template
instantiation is deferred.
---
Full diff: https://github.com/llvm/llvm-project/pull/103039.diff
5 F
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/106552
None
>From 5d313ed583f9ca8209fbdc04ecb0fd8174e74239 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Thu, 29 Aug 2024 13:46:18 +0200
Subject: [PATCH] [clang][bytecode] Add InitLinkScope for t
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/106552.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/Compiler.cpp (+1-3)
- (modified) clang/test/AST/ByteCode/records.cpp (+6)
```
@@ -257,7 +257,7 @@ General purpose options
A semicolon list of arguments to pass when running the libc++ benchmarks
using the
``check-cxx-benchmarks`` rule. By default we run the benchmarks for a very
short amount of time,
- since the primary use of ``check-cxx-benchma
https://github.com/AaronBallman approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/105727
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-08-29T15:22:59+02:00
New Revision: df11ee213e43ae373d1357939cf14ea37d547110
URL:
https://github.com/llvm/llvm-project/commit/df11ee213e43ae373d1357939cf14ea37d547110
DIFF:
https://github.com/llvm/llvm-project/commit/df11ee213e43ae373d1357939cf14ea37d547110.diff
L
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/106529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/91007
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
My concerns have been addressed, but there's still a small open question on the
driver flag behavior though, so please give @MaskRay a chance to respond before
landing.
Otherwise, LGTM!
https://github.com/llvm/llvm-project/pull/91007
@@ -2900,7 +2900,7 @@ defm clangir : BoolFOption<"clangir",
PosFlag,
NegFlag LLVM
pipeline to compile">,
BothFlags<[], [ClangOption, CC1Option], "">>;
-def emit_cir : Flag<["-"], "emit-cir">, Visibility<[CC1Option]>,
+def emit_cir : Flag<["-"], "emit-cir">, Visibility<[C
@@ -1144,6 +1164,46 @@ bool Sema::CheckInstantiatedFunctionTemplateConstraints(
PointOfInstantiation, Satisfaction);
}
+bool Sema::CheckFunctionConstraintsWithoutInstantiation(
cor3ntin wrote:
This is very similar to `Chec
@@ -3876,6 +3864,41 @@ TemplateDeductionResult
Sema::FinishTemplateArgumentDeduction(
FD = const_cast(FDFriend);
Owner = FD->getLexicalDeclContext();
}
+
+ // C++20 [temp.deduct.general]p5: [DR2369]
+ // If the function template has associated constraints, those co
@@ -1841,9 +1872,71 @@ Decl *TemplateInstantiator::TransformDecl(SourceLocation
Loc, Decl *D) {
// template parameter.
}
+ if (auto *PVD = dyn_cast(D);
+ PVD && PVD->getDeclContext() == FunctionDCForParameterDeclInstantiation
&&
+ !ParameterInstantiationScop
@@ -5585,11 +5585,12 @@ bool Sema::CheckTemplateArgumentList(
ContextRAII Context(*this, NewContext);
CXXThisScopeRAII(*this, RD, ThisQuals, RD != nullptr);
-MultiLevelTemplateArgumentList MLTAL = getTemplateInstantiationArgs(
-Template, NewContext, /*Final
@@ -3876,6 +3864,41 @@ TemplateDeductionResult
Sema::FinishTemplateArgumentDeduction(
FD = const_cast(FDFriend);
Owner = FD->getLexicalDeclContext();
}
+
+ // C++20 [temp.deduct.general]p5: [DR2369]
cor3ntin wrote:
```suggestion
// C++20 [temp.de
@@ -3876,6 +3864,41 @@ TemplateDeductionResult
Sema::FinishTemplateArgumentDeduction(
FD = const_cast(FDFriend);
Owner = FD->getLexicalDeclContext();
}
+
+ // C++20 [temp.deduct.general]p5: [DR2369]
+ // If the function template has associated constraints, those co
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/102857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -13051,11 +13051,14 @@ class Sema final : public SemaBase {
/// instantiation arguments.
///
/// \param DC In the event we don't HAVE a declaration yet, we instead
provide
- /// the decl context where it will be created. In this case, the
`Innermost`
- /// shoul
@@ -1397,12 +1415,25 @@ namespace {
ArrayRef Unexpanded,
bool &ShouldExpand, bool &RetainExpansion,
std::optional &NumExpansions) {
- return getSema().CheckParameterPacksForE
@@ -13051,11 +13051,14 @@ class Sema final : public SemaBase {
/// instantiation arguments.
///
/// \param DC In the event we don't HAVE a declaration yet, we instead
provide
- /// the decl context where it will be created. In this case, the
`Innermost`
- /// shoul
https://github.com/cor3ntin commented:
Thanks!
I did a first pass on the review, I have a few questions
https://github.com/llvm/llvm-project/pull/102857
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
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 101acff2e5df21378b5764660afc0ec84ce6a1a6
f3e4787973daf1cd278a9615960bbb4b31d3c0c0 --e
joker-eph wrote:
LG!
https://github.com/llvm/llvm-project/pull/105745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yronglin created
https://github.com/llvm/llvm-project/pull/106558
The PR https://github.com/llvm/llvm-project/pull/105996 broke taking the
address of a vector:
compound-literal.c
```C
typedef int v4i32 __attribute((vector_size(16)));
v4i32 *y = &(v4i32){1,2,3,4};
```
That bec
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (yronglin)
Changes
The PR https://github.com/llvm/llvm-project/pull/105996 broke taking the
address of a vector:
compound-literal.c
```C
typedef int v4i32 __attribute((vector_size(16)));
v4i32 *y = &(v4i32){1,2,3,4};
```
That because
phoebewang wrote:
Should we add folding in X86InstrFMA3Info.cpp?
https://github.com/llvm/llvm-project/pull/101603
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3682,6 +3682,8 @@ static const struct Extension {
{"sve2-bitperm", {AArch64::FeatureSVE2BitPerm}},
{"sve2p1", {AArch64::FeatureSVE2p1}},
{"ls64", {AArch64::FeatureLS64}},
+{"ls64_v", {AArch64::FeatureLS64_V}},
+{"ls64_accdata", {AArch64::FeatureLS64_ACCD
https://github.com/yronglin edited
https://github.com/llvm/llvm-project/pull/106558
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AaronBallman wrote:
Can you explain what changes were made to address the various issues which
caused the earlier revert? I've tried diffing the PRs to see what the changes
are, but my git-fu is insufficient to the task. :-D
https://github.com/llvm/llvm-project/pull/105479
AaronBallman wrote:
> I did think about it more, and I think it would be more consistent to support
> the attribute on any sort of specializable entity, including functions -
> regardless of STL needs @AaronBallman @ldionne
Agreed; if we're adding a custom attribute, we might as well support i
https://github.com/jpienaar approved this pull request.
Nice!
https://github.com/llvm/llvm-project/pull/105745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jpienaar edited
https://github.com/llvm/llvm-project/pull/105745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1654,12 +1654,12 @@ void OperationFormat::genElementParser(FormatElement
*element, MethodBody &body,
dir->shouldBeQualified() ? qualifiedTypeParserCode : typeParserCode;
TypeSwitch(dir->getArg())
.Case([&](auto operand) {
-body << form
philnik777 wrote:
> > I did think about it more, and I think it would be more consistent to
> > support the attribute on any sort of specializable entity, including
> > functions - regardless of STL needs @AaronBallman @ldionne
>
> Agreed; if we're adding a custom attribute, we might as well s
@@ -3648,35 +3648,38 @@ PathDiagnosticPieceRef
MallocBugVisitor::VisitNode(const ExplodedNode *N,
return nullptr;
}
- // See if we're releasing memory while inlining a destructor
- // (or one of its callees). This turns on various common
- //
@@ -3648,35 +3648,38 @@ PathDiagnosticPieceRef
MallocBugVisitor::VisitNode(const ExplodedNode *N,
return nullptr;
}
- // See if we're releasing memory while inlining a destructor
- // (or one of its callees). This turns on various common
- //
@@ -3558,8 +3558,8 @@ PathDiagnosticPieceRef MallocBugVisitor::VisitNode(const
ExplodedNode *N,
// original reference count is positive, we should not report use-after-frees
NagyDonat wrote:
Update the comment lines above this to be a bit more general. (Lines
@@ -3648,35 +3648,38 @@ PathDiagnosticPieceRef
MallocBugVisitor::VisitNode(const ExplodedNode *N,
return nullptr;
}
- // See if we're releasing memory while inlining a destructor
- // (or one of its callees). This turns on various common
- //
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/104599
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,80 @@
+// RUN: %clang_analyze_cc1 -analyzer-checker=core,unix.Malloc -verify %s
+//
+
+typedef unsigned long size_t;
+
+typedef enum memory_order {
+ memory_order_relaxed = __ATOMIC_RELAXED,
+} memory_order;
+
+void *calloc(size_t, size_t);
+void free(void *);
+
+struc
https://github.com/NagyDonat commented:
I like the overall idea of this commit, but I noticed that it may be a step
backwards in some situation where there is a destructor on the stack and the
old code would've suppressed the result, but the new code doesn't suppress it.
(See inline comments f
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/104599
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/EugeneZelenko edited
https://github.com/llvm/llvm-project/pull/106350
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/EugeneZelenko commented:
Please mention changes in Release Notes.
https://github.com/llvm/llvm-project/pull/106350
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -74,6 +77,40 @@ Both macros have to be defined to suggest replacement
functions from *Annex K.*
``__STDC_WANT_LIB_EXT1__`` must be defined to ``1`` by the user **before**
including any system headers.
+Custom functions
+
+
+The options `CustomNormalFunction
AaronBallman wrote:
> > > I did think about it more, and I think it would be more consistent to
> > > support the attribute on any sort of specializable entity, including
> > > functions - regardless of STL needs @AaronBallman @ldionne
> >
> >
> > Agreed; if we're adding a custom attribute, w
https://github.com/mh4ck-Thales updated
https://github.com/llvm/llvm-project/pull/100321
>From b812b9c2a7b92edf5dab739eadff0295c2a1f631 Mon Sep 17 00:00:00 2001
From: Quentin Michaud
Date: Wed, 24 Jul 2024 09:54:53 +0200
Subject: [PATCH 1/2] First draft of wasm-opt warning
---
clang/lib/Drive
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 73ef397fcba35b7b4239c00bf3e0b4e689ca0add
a798b1fd7a7a6c7b782584021e5c32e52c3abffc --e
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/106081
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -58,14 +60,14 @@ void testFreeOpNew() {
void *p = operator new(0);
free(p);
// mismatch-warning@-1{{Memory allocated by 'operator new' should be
deallocated by 'delete', not 'free()'}}
-} // leak-warning{{Potential leak of memory pointed to by 'p'}}
+}
---
https://github.com/NagyDonat commented:
Thanks for the updates! I'm mostly satisfied with this change, but I also added
some other reviewers to give them a chance to react.
https://github.com/llvm/llvm-project/pull/106081
___
cfe-commits mailing list
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/105727
>From d8bed3f4db8056a6afa9bd7eae5d4a8361f83086 Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Thu, 22 Aug 2024 23:25:31 +0300
Subject: [PATCH 1/2] [Clang] prevent assertion failure when converting vectors
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/94243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1224,7 +1224,7 @@ TEST_P(ASTMatchersTest,
CastExpression_MatchesImplicitCasts) {
}
TEST_P(ASTMatchersTest, CastExpr_DoesNotMatchNonCasts) {
- if (GetParam().Language == Lang_C89 || GetParam().Language == Lang_C99) {
+ if (GetParam().isC()) {
AaronBallma
https://github.com/AaronBallman commented:
modulo things brought up by @Sirraide, this is looking good!
https://github.com/llvm/llvm-project/pull/94243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
a-tarasyuk wrote:
could someone merge this if there are no concerns? thanks.
https://github.com/llvm/llvm-project/pull/105727
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/5chmidti updated
https://github.com/llvm/llvm-project/pull/94243
>From 1ef1bc0bfebca8ca114a57ea2d5016418e36446d Mon Sep 17 00:00:00 2001
From: Julian Schmidt
Date: Sat, 1 Jun 2024 17:38:37 +0200
Subject: [PATCH 1/4] [clang][test] add TestLanguage.def to specify all tested
la
@@ -1654,12 +1654,12 @@ void OperationFormat::genElementParser(FormatElement
*element, MethodBody &body,
dir->shouldBeQualified() ? qualifiedTypeParserCode : typeParserCode;
TypeSwitch(dir->getArg())
.Case([&](auto operand) {
-body << form
https://github.com/5chmidti updated
https://github.com/llvm/llvm-project/pull/94243
>From 1ef1bc0bfebca8ca114a57ea2d5016418e36446d Mon Sep 17 00:00:00 2001
From: Julian Schmidt
Date: Sat, 1 Jun 2024 17:38:37 +0200
Subject: [PATCH 1/5] [clang][test] add TestLanguage.def to specify all tested
la
mh4ck-Thales wrote:
I implemented a proposition on how to proceed for correctly informing users of
wasm-opt problems:
- a warning for when no flags are passed (the default being wasm-opt enabled
for wasip1) and wasm-opt is not found
- an error for when --wasm-opt is explicitly passed but wasm-o
dklimkin wrote:
Race condition! Will update the other PR.
https://github.com/llvm/llvm-project/pull/106398
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mh4ck-Thales ready_for_review
https://github.com/llvm/llvm-project/pull/100321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dklimkin wrote:
Hi darkfeline,
Sorry I didn't see your PR and merged #106398 earlier. Please consider closing
this one.
https://github.com/llvm/llvm-project/pull/106305
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cg
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Quentin Michaud (mh4ck-Thales)
Changes
Add a warning when `wasm-opt` is requested with a flag (#95208) but is
not found in the path.
I'm using #77148 as reference on how to add a new warning but please
tell me if you can help in i
https://github.com/jurahul closed
https://github.com/llvm/llvm-project/pull/105745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Rahul Joshi
Date: 2024-08-29T08:00:25-07:00
New Revision: fc110202dffa06950716e0cc4535b07aaa2c439c
URL:
https://github.com/llvm/llvm-project/commit/fc110202dffa06950716e0cc4535b07aaa2c439c
DIFF:
https://github.com/llvm/llvm-project/commit/fc110202dffa06950716e0cc4535b07aaa2c439c.diff
L
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`llvm-clang-x86_64-win-fast` running on `as-builder-3` while building
`clang,llvm,mlir` at step 7 "test-build-unified-tree-check-llvm-unit".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/2/builds/5504
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/106141
>From 9b83df995dc5c1d95db63a6ad32ba612b8a52290 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Mon, 26 Aug 2024 14:51:29 -0600
Subject: [PATCH 1/3] [flang][Driver] Add support for -mllvm
-print-pipeline-p
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-ve-ninja` running on
`hpce-ve-main` while building `clang,llvm,mlir` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/12/builds/4733
Here is the relevant piece of the build
https://github.com/5chmidti updated
https://github.com/llvm/llvm-project/pull/94243
>From 1ef1bc0bfebca8ca114a57ea2d5016418e36446d Mon Sep 17 00:00:00 2001
From: Julian Schmidt
Date: Sat, 1 Jun 2024 17:38:37 +0200
Subject: [PATCH 1/7] [clang][test] add TestLanguage.def to specify all tested
la
tarunprabhu wrote:
>
> To unblock you, I propose the following:
>
> 1. Keep this as is, but add a comment saying that this should be a proper
> compiler option instead of a secret handshake like this. Similar comment
> should be added in Clang. I will happily approve that.
>
I have adde
@@ -177,6 +177,26 @@ static bool isLanguageDefinedBuiltin(const SourceManager
&SourceMgr,
return false;
}
+static bool isReservedAttrName(Preprocessor &PP, IdentifierInfo *II) {
a-tarasyuk wrote:
@Sirraide @AaronBallman In the context of avoiding hardcoded
https://github.com/necto created
https://github.com/llvm/llvm-project/pull/106568
As reported in
https://github.com/llvm/llvm-project/pull/105648#issuecomment-2317144635 commit
08ad8dc7154bf3ab79f750e6d5fb7df597c7601a
introduced a nullptr dereference in the case when store contains a binding t
https://github.com/a-tarasyuk edited
https://github.com/llvm/llvm-project/pull/106036
___
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
@llvm/pr-subscribers-clang
Author: Arseniy Zaostrovnykh (necto)
Changes
As reported in
https://github.com/llvm/llvm-project/pull/105648#issuecomment-2317144635 commit
08ad8dc7154bf3ab79f750e6d5fb7df597c7601a
introduced a nullptr
necto wrote:
> Hello,
>
> The following starts crashing with this patch:
>
> ```
> clang -cc1 -analyze -analyzer-checker=core bbi-98571.c
> ```
>
> Result:
>
> ```
> (...)
> ```
Thank you for the report!
Here is the fix: https://github.com/llvm/llvm-project/pull/106568
https://github.com/ll
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 8ae877a089d9c2ca5315d3b2e0144c4d9f82cf5c
71aae8d0cc96d389da95c2231b1145b7ffeb2132 --e
https://github.com/Xazax-hun approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/106568
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/necto updated
https://github.com/llvm/llvm-project/pull/106568
>From 71aae8d0cc96d389da95c2231b1145b7ffeb2132 Mon Sep 17 00:00:00 2001
From: Arseniy Zaostrovnykh
Date: Thu, 29 Aug 2024 16:39:12 +0200
Subject: [PATCH 1/2] [analyzer] Fix nullptr dereference for symbols from
po
zengdage wrote:
@nikic @ChuanqiXu9 I am sorry to bother you again. Do you think the attribute
'nofree' should be added to 'this' pointer argument in c++ instance method ?
I think the 'this' pointer pointed to current object and it's life time is
guaranteed to be longer than the instance meth
https://github.com/zengdage reopened
https://github.com/llvm/llvm-project/pull/106499
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/106568
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -126,3 +126,22 @@ void caller_for_nested_leaking() {
int *ptr = 0;
caller_mid_for_nested_leaking(&ptr);
}
+
+// This used to crash StackAddrEscapeChecker because
+// it features a symbol conj_$1{struct c *, LC1, S763, #1}
+// that has no origin region.
+// bbi-98571
+str
@@ -126,3 +126,22 @@ void caller_for_nested_leaking() {
int *ptr = 0;
caller_mid_for_nested_leaking(&ptr);
}
+
+// This used to crash StackAddrEscapeChecker because
+// it features a symbol conj_$1{struct c *, LC1, S763, #1}
+// that has no origin region.
+// bbi-98571
+str
https://github.com/steakhal requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/106568
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -126,3 +126,22 @@ void caller_for_nested_leaking() {
int *ptr = 0;
caller_mid_for_nested_leaking(&ptr);
}
+
+// This used to crash StackAddrEscapeChecker because
+// it features a symbol conj_$1{struct c *, LC1, S763, #1}
+// that has no origin region.
+// bbi-98571
+str
@@ -126,3 +126,22 @@ void caller_for_nested_leaking() {
int *ptr = 0;
caller_mid_for_nested_leaking(&ptr);
}
+
+// This used to crash StackAddrEscapeChecker because
+// it features a symbol conj_$1{struct c *, LC1, S763, #1}
+// that has no origin region.
+// bbi-98571
https://github.com/zengdage edited
https://github.com/llvm/llvm-project/pull/106499
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
101 - 200 of 416 matches
Mail list logo
