rniwa wrote:
Thanks for the review!
https://github.com/llvm/llvm-project/pull/135526
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -464,7 +464,8 @@ fatbinary(ArrayRef>
InputFiles,
} // namespace amdgcn
namespace generic {
-Expected clang(ArrayRef InputFiles, const ArgList &Args)
{
+Expected clang(ArrayRef InputFiles, const ArgList &Args,
+ bool HasSYCLOffloadKind = false) {
-
@@ -937,13 +961,47 @@ Expected> linkAndWrapDeviceFiles(
InputFiles.emplace_back(*FileNameOrErr);
}
+if (HasSYCLOffloadKind) {
+ // Link the remaining device files using the device linker.
+ auto OutputOrErr = linkDevice(InputFiles, LinkerArgs,
HasSYCLO
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/135573
>From 213a2377387d8fc032741e4f4ddf50f88fccaca5 Mon Sep 17 00:00:00 2001
From: Oleksandr Tarasiuk
Date: Tue, 15 Apr 2025 01:07:05 +0300
Subject: [PATCH] [Clang] enhance loop analysis to handle variable changes
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`clang-cmake-x86_64-avx512-linux` running on `avx512-intel64` while building
`clang,llvm` at step 7 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/133/builds/14505
Here is the releva
eaeltsin wrote:
Comparing bcanalyzer --dump outputs for non-deterministic pcms, I see a lot of
op values that differ by 1.
I wonder if this might be something like the mismatch of Read Write
UnsignedOrNone vs unsigned ...
https://github.com/llvm/llvm-project/pull/132401
_
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/135573
>From 1bd4c4727596ad6255dd867fd3f2c9386b911bb5 Mon Sep 17 00:00:00 2001
From: Oleksandr Tarasiuk
Date: Tue, 15 Apr 2025 01:07:05 +0300
Subject: [PATCH] [Clang] enhance loop analysis to handle variable changes
https://github.com/ojhunt updated
https://github.com/llvm/llvm-project/pull/135686
>From 62be33b3aa475a33d1c11679ed069eb2af981754 Mon Sep 17 00:00:00 2001
From: Oliver Hunt
Date: Mon, 14 Apr 2025 15:02:46 -0700
Subject: [PATCH] [Clang]Ensure correct handling of access control in P2719
diagnost
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (jeremyd2019)
Changes
Add a new Cygwin toolchain that just goes through the motions to initialize the
Generic_GCC base properly. This allows removing some old, almost certainly
wrong hard-coded paths from Lex/InitHeaderSearch.cpp
T
https://github.com/farzonl converted_to_draft
https://github.com/llvm/llvm-project/pull/135655
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -605,13 +605,11 @@ void BackendConsumer::UnsupportedDiagHandler(
// Context will be nullptr for IR input files, we will construct the diag
// message from llvm::DiagnosticInfoUnsupported.
- if (Context != nullptr) {
+ if (Context != nullptr)
Loc = getBestLocation
AlexVlx wrote:
> LLVM Buildbot has detected a new failure on builder
> `llvm-clang-x86_64-sie-ubuntu-fast` running on `sie-linux-worker` while
> building `clang,llvm` at step 6 "test-build-unified-tree-check-all".
>
> Full details are available at:
> https://lab.llvm.org/buildbot/#/builders/1
@@ -67,6 +72,166 @@ void CIRDialect::printType(Type type, DialectAsmPrinter
&os) const {
llvm::report_fatal_error("printer is missing a handler for this type");
}
+//===--===//
+// RecordType Definitions
+/
s-watanabe314 wrote:
@andykaylor I would appreciate your comments on this.
https://github.com/llvm/llvm-project/pull/132680
___
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-modules
Author: Mariya Podchishchaeva (Fznamznon)
Changes
Finding operator delete[] is still problematic, without it the extension is a
security hazard, so reverting until the problem with operator delete[] is
figured out.
---
Patch is 86.63 Ki
https://github.com/Fznamznon created
https://github.com/llvm/llvm-project/pull/135611
Finding operator delete[] is still problematic, without it the extension is a
security hazard, so reverting until the problem with operator delete[] is
figured out.
>From 460d4807104c925232def63b516853f9ee22
Fznamznon wrote:
The clang format concerns were in main branch before I came in, so I think they
can be ignored.
https://github.com/llvm/llvm-project/pull/135611
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
https://github.com/ckandeler created
https://github.com/llvm/llvm-project/pull/135620
None
>From 6fb676cec0f907658339708ee17a9e73ea8b2cc7 Mon Sep 17 00:00:00 2001
From: Christian Kandeler
Date: Mon, 14 Apr 2025 14:13:41 +0200
Subject: [PATCH] [clangd] Support operators new and delete in
textD
AaronBallman wrote:
> This warning creates issues under Windows, where reinterpret-casting from
> FARPROC to the actual function type is common.
>
> #92738
> [boostorg/interprocess#259](https://github.com/boostorg/interprocess/issues/259)
>
> Reinterpret-casting a function pointer to another
zmodem wrote:
> This causes the whole libc++ CI to fail, since we're not building against a
> compiler built from current trunk.
Can you provide an example, such as a link to a failing build, or an
explanation of the failure? I found
https://buildkite.com/llvm-project/libcxx-ci/builds?created
https://github.com/zmodem closed
https://github.com/llvm/llvm-project/pull/135611
___
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: Christian Kandeler (ckandeler)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/135620.diff
2 Files Affected:
- (modified) clang-tools-extra/clangd/unittests/XRefsTests.cpp (+9-1)
- (modified) clang/lib/Index/IndexBody
Fznamznon wrote:
I'm working on a revert of the extension.
https://github.com/llvm/llvm-project/pull/133950
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-aarch64-sve-vla`
running on `linaro-g3-01` while building `clang` at step 7 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/17/builds/7273
Here is the relevant piece of the buil
https://github.com/mpark closed https://github.com/llvm/llvm-project/pull/135147
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -27,6 +27,8 @@
extern "C" {
#endif
+#if !defined(__CUDA_ARCH__)
+
rnk wrote:
> If we have to define them on non-windows targets anyways, perhaps a better
> fix is to let clang define them everywhere, and avoid this special case in
> the headers.
Yes, we
https://github.com/erichkeane approved this pull request.
Woops! Sorry, I thought I had already approved this with teh nit about the
tests. Sorry about that!
https://github.com/llvm/llvm-project/pull/134038
___
cfe-commits mailing list
cfe-commits@l
https://github.com/erichkeane approved this pull request.
This seems reasonable, but please give @AaronBallman a chance to respond, he
might have some insight as to why we shouldn't.
https://github.com/llvm/llvm-project/pull/135561
___
cfe-commits mai
https://github.com/AaronBallman approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/134398
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -870,7 +873,8 @@ def NSReturnsMismatch : DiagGroup<"nsreturns-mismatch">;
def IndependentClassAttribute : DiagGroup<"IndependentClass-attribute">;
def UnknownAttributes : DiagGroup<"unknown-attributes">;
-def IgnoredAttributes : DiagGroup<"ignored-attributes">;
+def Ignored
pdimov wrote:
Since you aren't diagnosing reinterpret-casting a function pointer from
`void*`, because this would warn on idiomatic POSIX code (`dlsym` returns
`void*`), it seemed to me that you'd be interested in not warning on idiomatic
Windows code (`GetProcAddress` returns `FARPROC`.)
But
@@ -43,7 +43,10 @@ enum NVVMMemorySpace {
/// Tensor memory space identifier.
/// Tensor memory is available only in arch-accelerated
/// variants from sm100 onwards.
- kTensorMemorySpace = 6
+ kTensorMemorySpace = 6,
+ /// Distributed shared memory space identifier.
+
https://github.com/tclin914 updated
https://github.com/llvm/llvm-project/pull/135110
>From 1615cb987f60d8c6123f7c95bc7bd7f22d897ea1 Mon Sep 17 00:00:00 2001
From: Jim Lin
Date: Wed, 9 Apr 2025 09:44:47 +0800
Subject: [PATCH 1/8] [RISCV] Add Andes XAndesperf (Andes Performance)
extension.
The
@@ -0,0 +1,258 @@
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
UTC_ARGS: --version 5
+; RUN: llc < %s -o - -mcpu=sm_90 -march=nvptx64 -mattr=+ptx80 | FileCheck %s
+; RUN: %if ptxas-12.0 %{ llc < %s -mtriple=nvptx64 -mcpu=sm_90 -mattr=+ptx80|
%pt
https://github.com/s-perron updated
https://github.com/llvm/llvm-project/pull/127675
>From 81c31fcdee28482ceea703064b0103eccb2a93ad Mon Sep 17 00:00:00 2001
From: Steven Perron
Date: Wed, 12 Feb 2025 15:45:32 -0500
Subject: [PATCH 1/2] [HLSL] Use hlsl_device address space for getpointer.
We ad
@@ -751,13 +751,14 @@
BuiltinTypeDeclBuilder::addHandleAccessFunction(DeclarationName &Name,
using PH = BuiltinTypeMethodBuilder::PlaceHolder;
QualType ElemTy = getHandleElementType();
- // TODO: Map to an hlsl_device address space.
- QualType ElemPtrTy = AST.getPointer
@@ -89,12 +89,12 @@ RESOURCE Buffer;
// CHECK-SAME{LITERAL}: [[hlsl::raw_buffer]]
// CHECK-SAME{LITERAL}: [[hlsl::contained_type(element_type)]]
-// CHECK-SUBSCRIPT: CXXMethodDecl {{.*}} operator[] 'const element_type
&(unsigned int) const'
+// CHECK-SUBSCRIPT: CXXMethodDecl
@@ -0,0 +1,17 @@
+// RUN: %clang_cc1 -verify -fsyntax-only %s
+
+int __attribute__((not_tail_called)) foo1(int a) {
+return a + 1;
+}
+
+
+int foo2(int a) {
+[[clang::musttail]]
+return foo1(a); // expected-error {{cannot perform a tail call to
function 'foo1' bec
Author: Matheus Izvekov
Date: 2025-04-14T10:44:25-03:00
New Revision: 10a15024212fe9ac06adad7daad2c1bb510a612c
URL:
https://github.com/llvm/llvm-project/commit/10a15024212fe9ac06adad7daad2c1bb510a612c
DIFF:
https://github.com/llvm/llvm-project/commit/10a15024212fe9ac06adad7daad2c1bb510a612c.dif
Author: Jack Styles
Date: 2025-04-14T08:15:42+01:00
New Revision: 53cd5cfc675dad1bf6bc820a72e0eaa72a8909e7
URL:
https://github.com/llvm/llvm-project/commit/53cd5cfc675dad1bf6bc820a72e0eaa72a8909e7
DIFF:
https://github.com/llvm/llvm-project/commit/53cd5cfc675dad1bf6bc820a72e0eaa72a8909e7.diff
L
@@ -67,6 +72,166 @@ void CIRDialect::printType(Type type, DialectAsmPrinter
&os) const {
llvm::report_fatal_error("printer is missing a handler for this type");
}
+//===--===//
+// RecordType Definitions
+/
@@ -738,6 +738,15 @@ void TextNodeDumper::Visit(const APValue &Value, QualType
Ty) {
else if (const auto *BE = B.dyn_cast()) {
OS << BE->getStmtClassName() << ' ';
dumpPointer(BE);
+} else if (B.is()) {
tbaederr wrote:
```suggestion
}
https://github.com/YLChenZ updated
https://github.com/llvm/llvm-project/pull/135178
>From 1119ecf764ecd94c12966758964cb486709a441d Mon Sep 17 00:00:00 2001
From: YLChenZ
Date: Thu, 10 Apr 2025 21:20:58 +0800
Subject: [PATCH 1/4] [clang][ast]: Add DynamicAllocLValue and TypeInfoLValue
support t
https://github.com/davemgreen edited
https://github.com/llvm/llvm-project/pull/130623
___
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 cpp,h --
clang/include/clang/Format/Format.h clang/lib/Forma
@@ -419,6 +419,9 @@ Bug Fixes to Attribute Support
- No longer crashing on ``__attribute__((align_value(N)))`` during template
instantiation when the function parameter type is not a pointer or reference.
(#GH26612)
+- The ``+nosimd`` attribute is now fully supported for AA
pdimov wrote:
This warning creates issues under Windows, where reinterpret-casting from
FARPROC to the actual function type is common.
https://github.com/llvm/llvm-project/pull/92738
https://github.com/boostorg/interprocess/issues/259
Reinterpret-casting a function pointer to another function
Author: Mariya Podchishchaeva
Date: 2025-04-14T14:17:36+02:00
New Revision: 88d0b0835d030635c5d08c9a9754c21b5ac00be9
URL:
https://github.com/llvm/llvm-project/commit/88d0b0835d030635c5d08c9a9754c21b5ac00be9
DIFF:
https://github.com/llvm/llvm-project/commit/88d0b0835d030635c5d08c9a9754c21b5ac00b
https://github.com/virginia-cangelosi updated
https://github.com/llvm/llvm-project/pull/130127
>From 3b94dd78f741574cef32798bfb863f39bc190eec Mon Sep 17 00:00:00 2001
From: Virginia Cangelosi
Date: Thu, 6 Mar 2025 13:38:19 +
Subject: [PATCH 1/3] [Clang][llvm] Implement fp8 FMOP4A intrinsics
AaronBallman wrote:
> @AaronBallman do you have any further requests or changes for this PR?
The only outstanding issue I know of is:
https://github.com/llvm/llvm-project/pull/100830/files#r2039375386
https://github.com/llvm/llvm-project/pull/100830
https://github.com/AaronBallman updated
https://github.com/llvm/llvm-project/pull/135407
>From c7e0132617ab01c12b393876b39381171996b793 Mon Sep 17 00:00:00 2001
From: Aaron Ballman
Date: Fri, 11 Apr 2025 13:03:07 -0400
Subject: [PATCH 1/3] Disable -fdollars-in-identifiers by default
Clang used
AaronBallman wrote:
> If we're going to do this, I think we need better diagnostics. Just straight
> disabling this is going to give very confusing diagnostics to anyone actually
> using dollar-signs in identifiers.
>
> Some ideas:
>
> * We can give a warning if we see a "$" adjacent to a
https://github.com/Stylie777 commented:
Thanks @davemgreen for the review.
https://github.com/llvm/llvm-project/pull/130623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -419,6 +419,9 @@ Bug Fixes to Attribute Support
- No longer crashing on ``__attribute__((align_value(N)))`` during template
instantiation when the function parameter type is not a pointer or reference.
(#GH26612)
+- The ``+nosimd`` attribute is now fully supported for AA
https://github.com/YLChenZ edited
https://github.com/llvm/llvm-project/pull/135178
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman commented:
Do we need a deprecation period for this change? This will break anyone using
the old names in scripts and whatnot.
Also, the changes will need something in the release notes so users know about
the new names.
https://github.com/llvm/llvm-project/pul
https://github.com/mmha updated https://github.com/llvm/llvm-project/pull/133405
>From 13833779faad62f95ef3fc0e2de3ed9b7c44d2f5 Mon Sep 17 00:00:00 2001
From: Morris Hafner
Date: Fri, 11 Apr 2025 17:46:00 +0200
Subject: [PATCH 1/2] [CIR] Upstream SelectOp and ShiftOp
Since SelectOp will only ge
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-remote-linux-ubuntu`
running on `as-builder-9` while building `clang` at step 16
"test-check-lldb-api".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/195/builds/7586
Here is the relevant piece
bwyma wrote:
@t-tye Would you mind reviewing this patch from a debug perspective?
Potentially multiple subprograms with the same source name, same source
correlation in different routines, missing artificial or trampoline attributes,
stepping between functions works as expected, etc. Your inpu
https://github.com/a-tarasyuk approved this pull request.
https://github.com/llvm/llvm-project/pull/134398
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -717,6 +717,12 @@ bool Sema::checkMustTailAttr(const Stmt *St, const Attr
&MTA) {
return false;
}
+ if (const FunctionDecl *CalleeDecl = CE->getDirectCallee();
+ CalleeDecl && CalleeDecl->hasAttr()) {
+Diag(St->getBeginLoc(), diag::err_musttail_mismatch) <<
@@ -67,6 +72,166 @@ void CIRDialect::printType(Type type, DialectAsmPrinter
&os) const {
llvm::report_fatal_error("printer is missing a handler for this type");
}
+//===--===//
+// RecordType Definitions
+/
https://github.com/Stylie777 updated
https://github.com/llvm/llvm-project/pull/130623
>From 978d9a153f9c5a98f6c20b94281f1acd02ecae4e Mon Sep 17 00:00:00 2001
From: Jack Styles
Date: Fri, 7 Mar 2025 15:51:34 +
Subject: [PATCH 1/8] [NFC][ARM] Split SIMD identifier away from MVE
Previously, t
Stylie777 wrote:
I have rebased this now after #134612 has been merged. This PR is only now
looking at enabling +nosimd for ARM targets.
https://github.com/llvm/llvm-project/pull/130623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://l
https://github.com/Stylie777 edited
https://github.com/llvm/llvm-project/pull/130623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/virginia-cangelosi updated
https://github.com/llvm/llvm-project/pull/130127
>From 3b94dd78f741574cef32798bfb863f39bc190eec Mon Sep 17 00:00:00 2001
From: Virginia Cangelosi
Date: Thu, 6 Mar 2025 13:38:19 +
Subject: [PATCH 1/2] [Clang][llvm] Implement fp8 FMOP4A intrinsics
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Yingwei Zheng (dtcxzyw)
Changes
Needed by https://github.com/llvm/llvm-project/pull/130742.
---
Full diff: https://github.com/llvm/llvm-project/pull/135602.diff
3 Files Affected:
- (modified) clang/test/CodeGenOpenCL/spir32_target.cl (
@@ -0,0 +1,180 @@
+
+// RUN: %clang_cc1 -fsyntax-only -Wms-bitfield-compatibility -verify -triple
armv8 -std=c++23 %s
+// RUN: %clang_cc1 -fsyntax-only -DMS_BITFIELDS -mms-bitfields
-verify=msbitfields -triple armv8-apple-macos10.15 -std=c++23 %s
+
+// msbitfields-no-diagnostics
https://github.com/dtcxzyw created
https://github.com/llvm/llvm-project/pull/135602
Needed by https://github.com/llvm/llvm-project/pull/130742.
>From e58df222ea5a12e302094b4fc77b0e77cbbc3e60 Mon Sep 17 00:00:00 2001
From: Yingwei Zheng
Date: Mon, 14 Apr 2025 17:18:09 +0800
Subject: [PATCH] [C
@@ -738,6 +738,15 @@ void TextNodeDumper::Visit(const APValue &Value, QualType
Ty) {
else if (const auto *BE = B.dyn_cast()) {
OS << BE->getStmtClassName() << ' ';
dumpPointer(BE);
+} else if (B.is()) {
YLChenZ wrote:
Done.
https://github
MrSidims wrote:
> This is the lowering to the target.
My glossary might not be lacking some definitions, but what I really meant by
lowering is:
https://github.com/llvm/llvm-project/blob/main/llvm/lib/Target/AMDGPU or
https://github.com/llvm/llvm-project/tree/main/llvm/lib/Target/SPIRV .
I d
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
@@ -67,6 +72,166 @@ void CIRDialect::printType(Type type, DialectAsmPrinter
&os) const {
llvm::report_fatal_error("printer is missing a handler for this type");
}
+//===--===//
+// RecordType Definitions
+/
@@ -0,0 +1,160 @@
+// NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
UTC_ARGS: --version 5
+// REQUIRES: aarch64-registered-target
+// RUN: %clang_cc1 -triple aarch64 -target-feature +sme-f8f32 -target-feature
+sme-f8f16 -target-feature +sme-mop4 -tar
@@ -3593,6 +3578,31 @@ class sme_tmopa_32b opc, RegisterOperand zn_ty,
RegisterOperand zm_ty, s
let Constraints = "$ZAda = $_ZAda";
}
+multiclass sme_tmopa_16b opc, RegisterOperand zn_ty, RegisterOperand
zm_ty, ValueType vt, string mnemonic, SDPatternOperator intrinsic,
l
@@ -3593,6 +3578,31 @@ class sme_tmopa_32b opc, RegisterOperand zn_ty,
RegisterOperand zm_ty, s
let Constraints = "$ZAda = $_ZAda";
}
+multiclass sme_tmopa_16b opc, RegisterOperand zn_ty, RegisterOperand
zm_ty, ValueType vt, string mnemonic, SDPatternOperator intrinsic,
l
frasercrmck wrote:
Note I'm in the process of introducing elementwise clz/ctz builtins which would
accomplish this and achieve vector intrinsics. See
https://github.com/llvm/llvm-project/pull/131995. It might be worth waiting for
that change instead?
https://github.com/llvm/llvm-project/pull/
https://github.com/frasercrmck closed
https://github.com/llvm/llvm-project/pull/134489
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/135573
>From 05125bafc45755624973feba0f4e6c2050fad36f Mon Sep 17 00:00:00 2001
From: Oleksandr Tarasiuk
Date: Mon, 14 Apr 2025 01:37:05 +0300
Subject: [PATCH] [Clang] enhance loop analysis to handle variable changes
https://github.com/tbaederr approved this pull request.
https://github.com/llvm/llvm-project/pull/135178
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Wenju He
Date: 2025-04-14T10:27:48+01:00
New Revision: cbda72a5474112e6bc62d5c5978ada7581e80dad
URL:
https://github.com/llvm/llvm-project/commit/cbda72a5474112e6bc62d5c5978ada7581e80dad
DIFF:
https://github.com/llvm/llvm-project/commit/cbda72a5474112e6bc62d5c5978ada7581e80dad.diff
LOG:
https://github.com/frasercrmck approved this pull request.
LGTM, thanks
https://github.com/llvm/llvm-project/pull/134489
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,15 @@
+//===--===//
+//
+// 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: Apac
zmodem wrote:
> Thanks! Unfortunately it looks like the fix introduced a regression of
> #134265 on Windows. Repro:
(For reference, this is https://crbug.com/41261 on our side.)
The "deleted operator delete[]" is coming from here:
https://source.chromium.org/chromium/chromium/src/+/main:v
ycongal-smile wrote:
I received 2 failing builds:
- LLVM Buildbot: builder sanitizer-x86_64-linux-bootstrap-asan build 7561
- https://lab.llvm.org/buildbot/#/builders/52/builds/7561
- LLVM Buildbot: builder sanitizer-x86_64-linux-fast build 10441
- https://lab.llvm.org/buildbot/#/builders/169
@@ -870,7 +873,8 @@ def NSReturnsMismatch : DiagGroup<"nsreturns-mismatch">;
def IndependentClassAttribute : DiagGroup<"IndependentClass-attribute">;
def UnknownAttributes : DiagGroup<"unknown-attributes">;
-def IgnoredAttributes : DiagGroup<"ignored-attributes">;
+def Ignored
@@ -759,6 +762,46 @@ LogicalResult cir::BinOp::verify() {
return mlir::success();
}
+//===--===//
+// ShiftOp
+//===--===//
+LogicalResult
pdimov wrote:
I get the warning for your code.
```
1>testbed2022.cpp(8,15): warning : cast from 'FARPROC' (aka 'long long (*)()')
to 'PGNSI' (aka 'void (*)(_SYSTEM_INFO *)') converts to incompatible function
type [-Wcast-function-type-mismatch]
```
Make sure you're passing /W4 to clang-cl.
htt
https://github.com/jhuber6 approved this pull request.
https://github.com/llvm/llvm-project/pull/134476
___
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-ir
Author: Acim Maravic (Acim-Maravic)
Changes
Added builtin and intrinsic for v_cvt_pk_norm_i16_f16 and v_cvt_pk_norm_u16_f16
---
Full diff: https://github.com/llvm/llvm-project/pull/135631.diff
8 Files Affected:
- (modified) clang/include/clan
@@ -0,0 +1,17 @@
+// RUN: %clang_cc1 -verify -fsyntax-only %s
+
+int __attribute__((not_tail_called)) foo1(int a) {
+return a + 1;
+}
+
+
+int foo2(int a) {
+[[clang::musttail]]
+return foo1(a); // expected-error {{cannot perform a tail call to
function 'foo1' bec
@@ -0,0 +1,17 @@
+// RUN: %clang_cc1 -verify -fsyntax-only %s
+
+int __attribute__((not_tail_called)) foo1(int a) {
+return a + 1;
+}
+
+
+int foo2(int a) {
+[[clang::musttail]]
+return foo1(a); // expected-error {{cannot perform a tail call to
function 'foo1' bec
@@ -717,6 +717,12 @@ bool Sema::checkMustTailAttr(const Stmt *St, const Attr
&MTA) {
return false;
}
+ if (const FunctionDecl *CalleeDecl = CE->getDirectCallee();
+ CalleeDecl && CalleeDecl->hasAttr()) {
+Diag(St->getBeginLoc(), diag::err_musttail_mismatch) <<
https://github.com/MacroModel created
https://github.com/llvm/llvm-project/pull/135643
when Ty.isDependentType() is true, it will crash here:
I encountered it while compiling my own project, which may not be the right
solution, but prevents crashes
```bash
Starting program: /home/MacroModel/l
@@ -0,0 +1,17 @@
+// RUN: %clang_cc1 -verify -fsyntax-only %s
+
+int __attribute__((not_tail_called)) foo1(int a) {
+return a + 1;
+}
+
+
+int foo2(int a) {
+[[clang::musttail]]
+return foo1(a); // expected-error {{cannot perform a tail call to
function 'foo1' bec
@@ -0,0 +1,17 @@
+// RUN: %clang_cc1 -verify -fsyntax-only %s
+
+int __attribute__((not_tail_called)) foo1(int a) {// expected-note
{{'not_tail_called' attribute here prevents being called as a tail call}}
+return a + 1;
+}
+
+
+int foo2(int a) {
+[[clang::musttail]]
+
https://github.com/dtcxzyw edited
https://github.com/llvm/llvm-project/pull/135649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bogner closed
https://github.com/llvm/llvm-project/pull/134570
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Justin Bogner
Date: 2025-04-14T10:34:15-07:00
New Revision: da17ced11b1cf44b433cb2b850978df4b6bff279
URL:
https://github.com/llvm/llvm-project/commit/da17ced11b1cf44b433cb2b850978df4b6bff279
DIFF:
https://github.com/llvm/llvm-project/commit/da17ced11b1cf44b433cb2b850978df4b6bff279.diff
Author: Ryosuke Niwa
Date: 2025-04-14T10:27:20-07:00
New Revision: 3e644859470e9ba77b8365cc7e2b5234492f4c30
URL:
https://github.com/llvm/llvm-project/commit/3e644859470e9ba77b8365cc7e2b5234492f4c30
DIFF:
https://github.com/llvm/llvm-project/commit/3e644859470e9ba77b8365cc7e2b5234492f4c30.diff
201 - 300 of 461 matches
Mail list logo