https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/137159
>From a489e3957eeb23fc38758c9a6670a3abd082321d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Thu, 24 Apr 2025 13:19:25 +0200
Subject: [PATCH] [clang][bytecode] Diagnose comparing pointers t
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: kadir çetinkaya (kadircet)
Changes
Previously we would defer evaluation of CLEs until LValue to RValue
conversions, which would result in creating values within wrong scope
and triggering use-after-frees.
This patch instead eagerly evaluat
https://github.com/AaronBallman approved this pull request.
The changes LGTM, so if we don't hear back from the folks we've pinged in the
next few days, I'd say this is safe to land.
https://github.com/llvm/llvm-project/pull/132984
___
cfe-commits mai
mstorsjo wrote:
This change makes Clang produce warnings when building Clang itself; warnings
looking like this:
```
/home/martin/code/llvm-project/clang/include/clang/Basic/LangOptions.def:352:1:
warning: bit-field 'FPEvalMethod' is not wide enough to store all enumerators
of preferred type '
@@ -3990,6 +3990,13 @@ def LocksExcluded : InheritableAttr {
let Documentation = [Undocumented];
}
+def ReentrantCapability : InheritableAttr {
+ let Spellings = [Clang<"reentrant_capability">];
+ let Subjects = SubjectList<[Record, TypedefName]>;
+ let Documentation = [U
https://github.com/anutosh491 updated
https://github.com/llvm/llvm-project/pull/136404
>From 87ca1f6992d1413d1d2b2e0d230b4f41e1979fef Mon Sep 17 00:00:00 2001
From: anutosh491
Date: Fri, 18 Apr 2025 18:45:00 +0530
Subject: [PATCH 1/2] Fix cuda flag with clang-repl
---
clang/include/clang/Inte
https://github.com/anutosh491 updated
https://github.com/llvm/llvm-project/pull/136404
>From 6c64e64c4a3b56f50808cae244b29da1231525f1 Mon Sep 17 00:00:00 2001
From: anutosh491
Date: Fri, 18 Apr 2025 18:45:00 +0530
Subject: [PATCH] Fix cuda flag with clang-repl
---
clang/include/clang/Interpre
anutosh491 wrote:
Making a note of the major changes below !
https://github.com/llvm/llvm-project/pull/136404
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AlexVlx approved this pull request.
LGTM, thanks! FWIW, perhaps it'd be worthwhile (as a follow-up, not here) to
simply make the DefIsGen AS map correct/valid, and replace the dummy bits?
https://github.com/llvm/llvm-project/pull/136753
___
=?utf-8?q?Bal=C3=A1zs_K=C3=A9ri?=
Message-ID:
In-Reply-To:
@@ -3,7 +3,7 @@
bugprone-invalid-enum-default-initialization
-Detect default initialization (to 0) of variables with `enum` type where
+Detect default initialization (to
=?utf-8?q?Balázs_Kéri?=
Message-ID:
In-Reply-To:
@@ -125,6 +125,12 @@ New checks
Finds potentially erroneous calls to ``reset`` method on smart pointers when
the pointee type also has a ``reset`` method.
+- New :doc:`bugprone-invalid-enum-default-initialization
+ ` che
@@ -609,8 +609,8 @@ def ConditionOp : CIR_Op<"condition", [
//===--===//
def YieldOp : CIR_Op<"yield", [ReturnLike, Terminator,
- ParentOneOf<["IfOp", "ScopeOp", "WhileOp",
-
@@ -2799,9 +2799,30 @@ CodeGenFunction::EmitLoadOfReference(LValue RefLVal,
llvm::LoadInst *Load =
Builder.CreateLoad(RefLVal.getAddress(), RefLVal.isVolatile());
CGM.DecorateInstructionWithTBAA(Load, RefLVal.getTBAAInfo());
- return makeNaturalAddressForPointer(Load
https://github.com/erichkeane commented:
Not the best one to do the full review here, but had 2 comments.
https://github.com/llvm/llvm-project/pull/137184
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
@@ -1,8 +1,8 @@
// RUN: %clang_cc1 -emit-llvm -triple %itanium_abi_triple -o - %s -O2 |
FileCheck %s
-// Make sure the call to b() doesn't get optimized out.
+// Make sure the call to b() is eliminated.
extern struct x {char& x,y;}y;
int b();
int a() { if (!&y.x) b();
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/137184
___
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: Morris Hafner (mmha)
Changes
This patch adds TernaryOp to CIR plus a pass that flattens the operator in
FlattenCFG.
This is the first PR out of (probably) 3 wrt. TernaryOp. I split the patches up
to make reviewing them easier. As such, t
@@ -0,0 +1,58 @@
+//===--- MisleadingSetterOfReferenceCheck.cpp -
clang-tidy-===//
+//
+// 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/mmha created https://github.com/llvm/llvm-project/pull/137184
This patch adds TernaryOp to CIR plus a pass that flattens the operator in
FlattenCFG.
This is the first PR out of (probably) 3 wrt. TernaryOp. I split the patches up
to make reviewing them easier. As such, this P
Author: erichkeane
Date: 2025-04-24T07:43:16-07:00
New Revision: d859cb68836191cfa469d0c951134b53ee31298e
URL:
https://github.com/llvm/llvm-project/commit/d859cb68836191cfa469d0c951134b53ee31298e
DIFF:
https://github.com/llvm/llvm-project/commit/d859cb68836191cfa469d0c951134b53ee31298e.diff
LO
steakhal wrote:
Do you have data about the analysis times per file, and per analysis entry
point?
Compared against the current llvm main, and also if this workaround would
restore the original running times before
https://github.com/llvm/llvm-project/commit/bb27d5e5c6b194a1440b8ac4e5ace68d0ee2
https://github.com/arsenm commented:
Even as a compiler implementation detail, libclc should not need to consider
the address space mapping.
There is a clang bug if there is different mangling. The itanium mangling
should be coming from the source type / original address space, not whatever I
https://github.com/arsenm edited
https://github.com/llvm/llvm-project/pull/137183
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
macurtis-amd wrote:
@nikic ping
https://github.com/llvm/llvm-project/pull/133301
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
IgnatSergeev wrote:
@bartlettroscoe,
> So is this really transforming the AST?
Well yeah, you are right, it replaces it with text, but what i ment, is that it
supports replacements with EditGenerators, which are really expressive (a few
words about them
https://clang.llvm.org/docs/ClangTrans
@@ -2523,6 +2523,20 @@ bool ExprEngine::replayWithoutInlining(ExplodedNode *N,
return true;
}
+/// Return the innermost location context which is inlined at `Node`, unless
+/// it's the top-level (entry point) location context.
+static const LocationContext *getInlinedLocati
@@ -399,3 +399,20 @@ namespace type_qualifier {
// CHECK: @_ZN14type_qualifier1gIPiEEvDTcmcvv_ELi1EE
template void g(int);
}
+
+namespace unresolved_template_specialization_type {
+ template struct enable_if {};
+ struct Foo {
+static const int value = true;
+ };
+
@@ -8197,6 +8197,16 @@ def err_address_space_qualified_new : Error<
def err_address_space_qualified_delete : Error<
"'delete' cannot delete objects of type %0 in address space '%1'">;
+def note_default_init_const_member : Note<
+ "member %0 declared 'const' here">;
+def war
https://github.com/jayfoad updated
https://github.com/llvm/llvm-project/pull/123072
>From f12511a0fd30c47ea08e6c126cae558215758183 Mon Sep 17 00:00:00 2001
From: Jay Foad
Date: Wed, 15 Jan 2025 13:34:41 +
Subject: [PATCH 01/16] Change API of getSuperClasses
---
llvm/include/llvm/TableGen/
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/137203
If the global is uninitialized.
>From 87377cd713004492ca9c989e1e7de806ba972ebf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Thu, 24 Apr 2025 17:46:49 +0200
Subject: [PATCH] [clang][byteco
https://github.com/ahatanak closed
https://github.com/llvm/llvm-project/pull/136771
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Akira Hatanaka
Date: 2025-04-24T08:47:29-07:00
New Revision: feaa5aa840dcda69bd4133536142be882f696114
URL:
https://github.com/llvm/llvm-project/commit/feaa5aa840dcda69bd4133536142be882f696114
DIFF:
https://github.com/llvm/llvm-project/commit/feaa5aa840dcda69bd4133536142be882f696114.diff
https://github.com/jakeegan updated
https://github.com/llvm/llvm-project/pull/129925
>From 072f4eca3825729d69d423d774c8a3298cb624a8 Mon Sep 17 00:00:00 2001
From: Jake Egan
Date: Wed, 5 Mar 2025 01:57:38 -0500
Subject: [PATCH 1/2] [clang] Implement address sanitizer on AIX (1/3)
The PR include
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
If the global is uninitialized.
---
Full diff: https://github.com/llvm/llvm-project/pull/137203.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/Interp.h (+1-1)
- (modified) clang/test/AST/ByteC
https://github.com/NagyDonat edited
https://github.com/llvm/llvm-project/pull/136720
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
NagyDonat wrote:
@balazs-benics-sonarsource Unfortunately our current CI (which runs the static
analyzer via CodeChecker) cannot provide file-level or entrypoint-level timing
information, but I do have full analysis runtime measurements (which I'll edit
into the table in my previous comment).
@@ -1660,6 +1660,10 @@ def err_omp_expected_colon : Error<"missing ':' in %0">;
def err_omp_missing_comma : Error< "missing ',' after %0">;
def err_omp_expected_context_selector
: Error<"expected valid context selector in %0">;
+def err_omp_unknown_clause
+: Error<"expe
@@ -2785,6 +2797,13 @@ StmtResult
Parser::ParseOpenMPDeclarativeOrExecutableDirective(
return Directive;
}
}
+ if (CKind == OMPC_otherwise) {
+// Check for 'otherwise' keyword.
+if (Tok.is(tok::identifier) &&
+Tok.getIde
@@ -48,14 +48,6 @@ if (LLVM_ENABLE_DOXYGEN)
set(bolt_doxygen_qhp_cust_filter_attrs "")
endif()
- option(LLVM_DOXYGEN_SVG
AaronBallman wrote:
Awesome, thanks!
https://github.com/llvm/llvm-project/pull/136843
@@ -13,20 +13,76 @@
#include "CIRGenCall.h"
#include "CIRGenFunction.h"
+#include "CIRGenFunctionInfo.h"
#include "clang/CIR/MissingFeatures.h"
using namespace clang;
using namespace clang::CIRGen;
-CIRGenFunctionInfo *CIRGenFunctionInfo::create(CanQualType resultType)
https://github.com/SLTozer updated
https://github.com/llvm/llvm-project/pull/136867
>From 90c09c8326077a1ba6797519bbefd014f32b5beb Mon Sep 17 00:00:00 2001
From: Stephen Tozer
Date: Wed, 23 Apr 2025 14:24:50 +0100
Subject: [PATCH 1/3] [Clang] Emit Fake Uses before musttail calls
Fixes the issu
https://github.com/kadircet updated
https://github.com/llvm/llvm-project/pull/137163
From 99fa9a1f5fb74401c790871e6eb1ce473d048d74 Mon Sep 17 00:00:00 2001
From: Kadir Cetinkaya
Date: Thu, 24 Apr 2025 11:12:00 +0200
Subject: [PATCH] [clang][CompundLiteralExpr] Don't defer evaluation for CLEs
P
https://github.com/kadircet edited
https://github.com/llvm/llvm-project/pull/137163
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet created
https://github.com/llvm/llvm-project/pull/137163
Previously we would defer evaluation of CLEs until LValue to RValue
conversions, which would result in creating values within wrong scope
and triggering use-after-frees.
This patch instead eagerly evaluates CLE
kadircet wrote:
Put together a new fix based on the discussions here in
https://github.com/llvm/llvm-project/pull/137163, PTAL.
@AaronBallman I am afraid this might no longer fix the crashes you're seeing
internally though, as the new fix is focused on handling of
`CompoundLiteralExpr`s, wher
https://github.com/T-Tie updated
https://github.com/llvm/llvm-project/pull/136694
>From e98beadf0009fa05339b964780e4dfe7c444fdd9 Mon Sep 17 00:00:00 2001
From: Tie
Date: Tue, 22 Apr 2025 12:25:32 +
Subject: [PATCH 1/2] Add Support for Ziccamoc
---
clang/test/Preprocessor/riscv-target-feat
https://github.com/AaronBallman updated
https://github.com/llvm/llvm-project/pull/137166
>From bb184fc38d13126f244615425cfefe7368ca71b8 Mon Sep 17 00:00:00 2001
From: Aaron Ballman
Date: Wed, 23 Apr 2025 13:40:44 -0400
Subject: [PATCH 1/6] Add -Wdefault-const-init
---
clang/docs/ReleaseNotes.
https://github.com/anutosh491 updated
https://github.com/llvm/llvm-project/pull/136404
>From 1f7205615f8d11c1b58e2a2760f85663f97767c5 Mon Sep 17 00:00:00 2001
From: anutosh491
Date: Fri, 18 Apr 2025 18:45:00 +0530
Subject: [PATCH 1/4] Fix cuda flag with clang-repl
---
clang/include/clang/Inte
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,h --
clang/test/AST/static-compound-literals.cpp clang/i
AaronBallman wrote:
Thanks! You're right, it may not address the issue we were seeing. Some of our
were also ending up with:
```
==3187048==WARNING: MemorySanitizer: use-of-uninitialized-value
#0 0x1f73c115 in clang::APValue::operator=(clang::APValue&&)
(/netbatch/donb2642835_00/runDir/
asb wrote:
> If you're planning on backporting this, should the release notes be added in
> a separate commit? I'm wondering if you'll end up with issues trying to
> cherry-pick it otherwise
I think we'd typically land a change in the most logical way for the main
branch and then handle unpic
https://github.com/anutosh491 edited
https://github.com/llvm/llvm-project/pull/136404
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/anutosh491 edited
https://github.com/llvm/llvm-project/pull/136404
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/balazske updated
https://github.com/llvm/llvm-project/pull/136823
From 4ce7497bb0dc89de3b9f139177c295291e7d3e9c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bal=C3=A1zs=20K=C3=A9ri?=
Date: Thu, 17 Apr 2025 17:36:03 +0200
Subject: [PATCH 1/2] [clang-tidy] Add check
'bugprone-inva
@@ -399,3 +399,20 @@ namespace type_qualifier {
// CHECK: @_ZN14type_qualifier1gIPiEEvDTcmcvv_ELi1EE
template void g(int);
}
+
+namespace unresolved_template_specialization_type {
+ template struct enable_if {};
+ struct Foo {
+static const int value = true;
+ };
+
https://github.com/steakhal approved this pull request.
As for the Static Analyzer, if the tests pass, we should be good.
Unique_ptr > shared_ptr for sure!
https://github.com/llvm/llvm-project/pull/132984
___
cfe-commits mailing list
cfe-commits@lists.
llvmbot wrote:
@llvm/pr-subscribers-backend-amdgpu
Author: Fraser Cormack (frasercrmck)
Changes
These were only being used when compiling with versions of clang older than
clang 6. As such they were essentially unsuppored and untested.
This somewhat simplifies the codebase, producing few
@@ -28,20 +28,21 @@ IncrementalCUDADeviceParser::IncrementalCUDADeviceParser(
std::unique_ptr DeviceInstance,
CompilerInstance &HostInstance,
llvm::IntrusiveRefCntPtr FS,
-llvm::Error &Err, const std::list &PTUs)
vgvassilev wrote:
Why removing
@@ -172,6 +154,19 @@ llvm::Error
IncrementalCUDADeviceParser::GenerateFatbinary() {
FatbinContent.append(PTXCode.begin(), PTXCode.end());
+ auto &PTU = PTUs.back();
vgvassilev wrote:
```suggestion
const PartialTranslationUnit &PTU = PTUs.back();
```
h
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/135251
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex ready_for_review
https://github.com/llvm/llvm-project/pull/137187
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Shafik Yaghmour
Date: 2025-04-24T08:15:25-07:00
New Revision: 72cc868c65b0641f23d1fb0518a8503c73ecdb5a
URL:
https://github.com/llvm/llvm-project/commit/72cc868c65b0641f23d1fb0518a8503c73ecdb5a
DIFF:
https://github.com/llvm/llvm-project/commit/72cc868c65b0641f23d1fb0518a8503c73ecdb5a.dif
@@ -760,8 +787,10 @@ std::unique_ptr Interpreter::GenModule() {
return nullptr;
}
-CodeGenerator *Interpreter::getCodeGen() const {
- FrontendAction *WrappedAct = Act->getWrapped();
+CodeGenerator *Interpreter::getCodeGen(IncrementalAction *Action) const {
+ if (!Action)
+
https://github.com/shafik closed
https://github.com/llvm/llvm-project/pull/137095
___
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: Nick Sarnie (sarnex)
Changes
Based on feedback from https://github.com/llvm/llvm-project/pull/136753, remove
the dummy values for OpenCL and make them match the zero default AS map.
---
Full diff: https://github.com/llvm/llvm-project/pull
@@ -2523,6 +2523,20 @@ bool ExprEngine::replayWithoutInlining(ExplodedNode *N,
return true;
}
+/// Return the innermost location context which is inlined at `Node`, unless
+/// it's the top-level (entry point) location context.
+static const LocationContext *getInlinedLocati
@@ -414,6 +414,20 @@ ANALYZER_OPTION(
"any target.",
true)
+ANALYZER_OPTION(
+bool, LegacyInliningPrevention, "legacy-inlining-prevention",
+"If enabled, the analyzer puts functions on a \"do not inline this\" list "
+"if it finds an execution path within t
@@ -2856,8 +2873,29 @@ void ExprEngine::processBranch(
// conflicts with the widen-loop analysis option (which is off by
// default). If we intend to support and stabilize the loop widening,
// we must ensure that it 'plays nicely' with this logic.
- if (
@@ -2570,21 +2584,24 @@ void ExprEngine::processCFGBlockEntrance(const
BlockEdge &L,
const ExplodedNode *Sink =
nodeBuilder.generateSink(Pred->getState(), Pred, &tag);
-// Check if we stopped at the top level function or not.
-// Root node shoul
https://github.com/anutosh491 updated
https://github.com/llvm/llvm-project/pull/136404
>From 6c64e64c4a3b56f50808cae244b29da1231525f1 Mon Sep 17 00:00:00 2001
From: anutosh491
Date: Fri, 18 Apr 2025 18:45:00 +0530
Subject: [PATCH 1/3] Fix cuda flag with clang-repl
---
clang/include/clang/Inte
@@ -414,6 +414,20 @@ ANALYZER_OPTION(
"any target.",
true)
+ANALYZER_OPTION(
+bool, LegacyInliningPrevention, "legacy-inlining-prevention",
+"If enabled, the analyzer puts functions on a \"do not inline this\" list "
+"if it finds an execution path within t
https://github.com/anutosh491 edited
https://github.com/llvm/llvm-project/pull/136404
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -50,25 +50,6 @@ IncrementalCUDADeviceParser::Parse(llvm::StringRef Input) {
if (!PTU)
return PTU.takeError();
- auto PTX = GeneratePTX();
anutosh491 wrote:
Yes absolutely !
Sorry for overlooking this. Made the change.
https://github.com/llvm/llvm-
@@ -2570,21 +2584,24 @@ void ExprEngine::processCFGBlockEntrance(const
BlockEdge &L,
const ExplodedNode *Sink =
nodeBuilder.generateSink(Pred->getState(), Pred, &tag);
-// Check if we stopped at the top level function or not.
-// Root node shoul
https://github.com/el-ev updated
https://github.com/llvm/llvm-project/pull/136854
>From fd1d37ebf0f1c4a23f573444f30730b0f65630bf Mon Sep 17 00:00:00 2001
From: Iris Shi <0...@owo.li>
Date: Wed, 23 Apr 2025 20:23:09 +0800
Subject: [PATCH 1/2] [CIR] Cleanup support for C functions
---
clang/lib/
https://github.com/AaronBallman commented:
I think the changes LG, but please wait for our CMake experts (@petrhosek and
@Ericson2314) to review before landing.
https://github.com/llvm/llvm-project/pull/136843
___
cfe-commits mailing list
cfe-commits@
@@ -569,7 +569,7 @@
CrossTranslationUnitContext::ASTLoader::loadFromDump(StringRef ASTDumpPath) {
return ASTUnit::LoadFromASTFile(
ASTDumpPath, CI.getPCHContainerOperations()->getRawReader(),
ASTUnit::LoadEverything, Diags, CI.getFileSystemOpts(),
- CI.getHe
@@ -0,0 +1,198 @@
+// RUN: %clang_analyze_cc1 -analyzer-checker=core,debug.ExprInspection
-verify=expected,default %s
+// RUN: %clang_analyze_cc1 -analyzer-checker=core,debug.ExprInspection
-analyzer-config legacy-inlining-prevention=false -verify=expected,disabled %s
+
+int get
alexfh wrote:
> I'm going to separate out the CUDA SPIRV regression fix as I need to fix some
> problems with this PR, will add you as a reviewer.
Do you have a timeline? It would be nice to get this fixed soon.
https://github.com/llvm/llvm-project/pull/135251
_
https://github.com/el-ev updated
https://github.com/llvm/llvm-project/pull/136854
>From fd1d37ebf0f1c4a23f573444f30730b0f65630bf Mon Sep 17 00:00:00 2001
From: Iris Shi <0...@owo.li>
Date: Wed, 23 Apr 2025 20:23:09 +0800
Subject: [PATCH 1/2] [CIR] Cleanup support for C functions
---
clang/lib/
@@ -233,6 +233,19 @@ int f8(int *p) {
// OGCG: %[[P2:.*]] = load ptr, ptr %[[P_PTR]], align 8
// OGCG: %[[STAR_P:.*]] = load i32, ptr %[[P2]], align 4
+
+void f9() {}
el-ev wrote:
I'm actually unsure what kind of new tests would be appropriate. Do you ha
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/137159
... with different access specifiers.
>From a4c3afca1ef1d029aea8c70d83195100e48b39e3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Thu, 24 Apr 2025 13:19:25 +0200
Subject: [PATCH] [clang][
kadircet wrote:
Followup for https://github.com/llvm/llvm-project/pull/118480
https://github.com/llvm/llvm-project/pull/137163
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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 cl,c,cpp --
clang/test/Sema/warn-default-const-init.c clang/
https://github.com/anutosh491 updated
https://github.com/llvm/llvm-project/pull/136404
>From 1f7205615f8d11c1b58e2a2760f85663f97767c5 Mon Sep 17 00:00:00 2001
From: anutosh491
Date: Fri, 18 Apr 2025 18:45:00 +0530
Subject: [PATCH 1/2] Fix cuda flag with clang-repl
---
clang/include/clang/Inte
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
... with different access specifiers.
---
Full diff: https://github.com/llvm/llvm-project/pull/137159.diff
4 Files Affected:
- (modified) clang/lib/AST/ByteCode/Interp.h (+13)
- (modified) clang/lib/AST/B
https://github.com/fangyi-zhou ready_for_review
https://github.com/llvm/llvm-project/pull/137182
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sarnex wrote:
Thanks for the feedback, I'll work to address it.
https://github.com/llvm/llvm-project/pull/135251
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Nick Sarnie
Date: 2025-04-24T14:20:13Z
New Revision: 52a96491e1e4e0d033e39fad87f49ccd871df41d
URL:
https://github.com/llvm/llvm-project/commit/52a96491e1e4e0d033e39fad87f49ccd871df41d
DIFF:
https://github.com/llvm/llvm-project/commit/52a96491e1e4e0d033e39fad87f49ccd871df41d.diff
LOG: [
sarnex wrote:
@alexfh The regression is fixed in
https://github.com/llvm/llvm-project/commit/52a96491e1e4e0d033e39fad87f49ccd871df41d
https://github.com/llvm/llvm-project/pull/135251
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://list
https://github.com/sarnex closed
https://github.com/llvm/llvm-project/pull/136753
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sarnex wrote:
Thanks for the review, and will do @AlexVlx!
https://github.com/llvm/llvm-project/pull/136753
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -8197,6 +8197,16 @@ def err_address_space_qualified_new : Error<
def err_address_space_qualified_delete : Error<
"'delete' cannot delete objects of type %0 in address space '%1'">;
+def note_default_init_const_member : Note<
+ "member %0 declared 'const' here">;
+def war
@@ -6496,6 +6496,17 @@ static bool canPerformArrayCopy(const InitializedEntity
&Entity) {
return false;
}
+static const FieldDecl *GetConstField(const RecordDecl *RD) {
erichkeane wrote:
Ah, I see! Would love an assert to that effect.
https://github.com/
https://github.com/erichkeane approved this pull request.
1 spelling nit, 1 request for an assert, else LGTM.
https://github.com/llvm/llvm-project/pull/137166
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
https://github.com/fangyi-zhou created
https://github.com/llvm/llvm-project/pull/137182
Per suggestion in
https://github.com/llvm/llvm-project/pull/128251#discussion_r2055916229, adding
a new helper function in `SValBuilder` to conjure a symbol when given a
`CallEvent`.
Tested manually (with
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Fangyi Zhou (fangyi-zhou)
Changes
Per suggestion in
https://github.com/llvm/llvm-project/pull/128251#discussion_r2055916229, adding
a new helper function in `SValBuilder` to conjure a symbol when given a
`CallEvent`.
Te
@@ -8197,6 +8197,16 @@ def err_address_space_qualified_new : Error<
def err_address_space_qualified_delete : Error<
"'delete' cannot delete objects of type %0 in address space '%1'">;
+def note_default_init_const_member : Note<
+ "member %0 declared 'const' here">;
+def war
@@ -139,22 +141,41 @@ class FactEntry : public CapabilityExpr {
bool declared() const { return Source == Declared; }
bool managed() const { return Source == Managed; }
- virtual void
- handleRemovalFromIntersection(const FactSet &FSet, FactManager &FactMan,
-
@@ -1217,11 +1217,13 @@ void
CGOpenMPRuntimeGPU::emitParallelCall(CodeGenFunction &CGF,
CGBuilderTy &Bld = CGF.Builder;
llvm::Value *NumThreadsVal = NumThreads;
llvm::Function *WFn = WrapperFunctionsMap[OutlinedFn];
+llvm::FunctionCallee RuntimeFn = OMPBuilder.
1 - 100 of 501 matches
Mail list logo
