https://github.com/joaosaffran updated
https://github.com/llvm/llvm-project/pull/109331
>From 50d21754119ac10c2ee2376ed8f79d12f73cd137 Mon Sep 17 00:00:00 2001
From: Joao Saffran
Date: Thu, 19 Sep 2024 00:13:51 +
Subject: [PATCH 1/3] Codegen builtin
---
clang/include/clang/Basic/Builtins.
@@ -4719,8 +4719,9 @@ class FunctionEffect {
NonBlocking = 1,
Sirraide wrote:
(github being confusing again; this is meant to reference the line above that
one in case that’s not obvious...)
https://github.com/llvm/llvm-project/pull/99656
_
@@ -12658,10 +12658,10 @@ This instruction requires several arguments:
the return value of the callee is returned to the caller's caller, even
if a void return type is in use.
- Both markers imply that the callee does not access allocas from the caller.
- The `
https://github.com/rnk commented:
Thanks for prioritizing this! I have triaged many internal issues about this.
https://github.com/llvm/llvm-project/pull/109943
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
https://github.com/rnk edited https://github.com/llvm/llvm-project/pull/109943
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Sirraide wrote:
ping
https://github.com/llvm/llvm-project/pull/85325
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ahatanak created
https://github.com/llvm/llvm-project/pull/110047
The new cast kinds are needed to distinguish between no-op conversions and
conversions from pointers to noexcept functions to pointers to functions
without noexcept as the latter can cause function pointers to
https://github.com/ahatanak closed
https://github.com/llvm/llvm-project/pull/110047
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -401,6 +401,194 @@ void SemaHLSL::DiagnoseAttrStageMismatch(
<< (AllowedStages.size() != 1) << join(StageStrings, ", ");
}
+template
+static void castVector(Sema &S, ExprResult &E, QualType &Ty, unsigned Sz) {
+ if (const auto *VTy = Ty->getAs())
+Ty = VTy->getE
https://github.com/bogner approved this pull request.
https://github.com/llvm/llvm-project/pull/108659
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane approved this pull request.
I don't see any reason to hold this up, and its waited long enough. So I'm
going to consider this 'baked' enough in review.
https://github.com/llvm/llvm-project/pull/85325
___
cfe-commits ma
Sirraide wrote:
Alright, I’ll merge main and then merge this once CI is done.
https://github.com/llvm/llvm-project/pull/85325
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide updated
https://github.com/llvm/llvm-project/pull/85325
>From 907210a3ad3d829a8e49a5c976d129f8653801bf Mon Sep 17 00:00:00 2001
From: Sirraide
Date: Thu, 14 Mar 2024 18:24:37 +0100
Subject: [PATCH 01/16] [Clang] Add `dump()` method for `Attr`
---
clang/include/clan
https://github.com/ahatanak created
https://github.com/llvm/llvm-project/pull/110048
The new cast kinds are needed to distinguish between no-op conversions and
conversions from pointers to noexcept functions to pointers to functions
without noexcept as the latter can cause function pointers to
https://github.com/joaosaffran updated
https://github.com/llvm/llvm-project/pull/109331
>From 50d21754119ac10c2ee2376ed8f79d12f73cd137 Mon Sep 17 00:00:00 2001
From: Joao Saffran
Date: Thu, 19 Sep 2024 00:13:51 +
Subject: [PATCH 1/3] Codegen builtin
---
clang/include/clang/Basic/Builtins.
@@ -0,0 +1,1572 @@
+//=== SemaFunctionEffects.cpp - Sema handling of function effects
-===//
+//
+// 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: A
https://github.com/jurahul updated
https://github.com/llvm/llvm-project/pull/110032
>From 934b0bd5934327429d0b878a9ece8a0c61e89400 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Wed, 25 Sep 2024 09:53:45 -0700
Subject: [PATCH] [LLVM][TableGen] Change SeachableTableEmitter to use const
Record
@@ -0,0 +1,1572 @@
+//=== SemaFunctionEffects.cpp - Sema handling of function effects
-===//
+//
+// 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: A
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/99656
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4719,8 +4719,9 @@ class FunctionEffect {
NonBlocking = 1,
dougsonos wrote:
Good idea. I was afraid of a lot of fallout but it was actually quite an easy
change to get rid of the sentinel value.
https://github.com/llvm/llvm-project/pull/99656
__
@@ -78,22 +78,19 @@ class ViolationSite {
// be inferred as holding that effect.
struct Violation {
FunctionEffect Effect;
- FunctionEffect
+ std::optional
CalleeEffectPreventingInference; // Only for certain IDs; can be None.
Sirraide wrote:
```sug
@@ -616,6 +626,62 @@ class LineJoiner {
return 1;
}
+ unsigned tryMergeNamespace(SmallVectorImpl::const_iterator
I,
+ SmallVectorImpl::const_iterator
E,
+ unsigned Limit) {
+if (Limit == 0)
g
@@ -616,6 +626,62 @@ class LineJoiner {
return 1;
}
+ unsigned tryMergeNamespace(SmallVectorImpl::const_iterator
I,
+ SmallVectorImpl::const_iterator
E,
+ unsigned Limit) {
+if (Limit == 0)
+ return 0;
+
Sirraide wrote:
Relevant pr: #85325
https://github.com/llvm/llvm-project/pull/110068
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide closed
https://github.com/llvm/llvm-project/pull/110068
___
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-armv8-quick` running
on `linaro-clang-armv8-quick` while building `clang` at step 5 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/154/builds/5052
Here is the relevant piece of
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `llvm-clang-x86_64-sie-win`
running on `sie-win-worker` while building `clang` at step 7
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/46/builds/5553
Here is the
kawashima-fj wrote:
Thanks for the PR.
Is there a symbolic link `flang-new` -> `flang`? We would like to have a
transitional period.
https://github.com/llvm/llvm-project/pull/110023
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists
dtcxzyw wrote:
> -fsanitize=address,fuzzer
I think it is not related to this patch. It only works with
`-fsanitize=builtin` or `-fsanitize=undefined`.
https://github.com/llvm/llvm-project/pull/104741
___
cfe-commits mailing list
cfe-commits@lists.ll
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/109302
>From cc2c798193722b3a537c76e74981ff767d064efa Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 19 Sep 2024 23:46:16 +0800
Subject: [PATCH 1/4] [clang-tidy][bugprone-posix-return] support integer
lite
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/108792
>From 795b3ae677210ff50f7710a0cf73d435889b68ae Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Mon, 16 Sep 2024 13:47:10 +0800
Subject: [PATCH] [clang-tidy] insert ``static`` keyword in correct position
f
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/109159
>From 4fc1d24c4ff22a8da22454aebe7053ea76419767 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 18 Sep 2024 23:26:02 +0800
Subject: [PATCH] [clang-tidy] loop convert can handle lambda init capture
Fix
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/110039
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ruiling commented:
With one trick mentioned inline, we can further reduce one instruction. In most
cases, we would have to introduce one scalar instruction in if-block and one
salu in flow-block compared with existing solution.
Block prologue is unclear concept and not well
@@ -1391,24 +1419,28 @@ define amdgpu_ps float @test_control_flow_0(<8 x i32>
inreg %rsrc, <4 x i32> inr
; GFX10-W32: ; %bb.0: ; %main_body
; GFX10-W32-NEXT:s_mov_b32 s12, exec_lo
; GFX10-W32-NEXT:s_wqm_b32 exec_lo, exec_lo
-; GFX10-W32-NEXT:s_mov_b32 s13, ex
@@ -446,8 +474,10 @@ define amdgpu_kernel void @add_i32_uniform(ptr
addrspace(1) %out, ptr addrspace(
; GFX11W64-NEXT:; implicit-def: $vgpr1
; GFX11W64-NEXT:s_delay_alu instid0(VALU_DEP_1) | instskip(NEXT) |
instid1(VALU_DEP_1)
; GFX11W64-NEXT:v_mbcnt_hi_u32_b32 v
https://github.com/ruiling edited
https://github.com/llvm/llvm-project/pull/108596
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -936,19 +978,20 @@ define amdgpu_kernel void @add_i32_varying_vdata(ptr
addrspace(1) %out, ptr addr
; GFX12W32-NEXT:s_cbranch_scc1 .LBB2_1
; GFX12W32-NEXT: ; %bb.2: ; %ComputeEnd
; GFX12W32-NEXT:v_mbcnt_lo_u32_b32 v1, exec_lo, 0
-; GFX12W32-NEXT:s_delay_alu ins
https://github.com/agozillon updated
https://github.com/llvm/llvm-project/pull/109907
>From 6e29de9836f49e49558e7810ec0a8084017c22b4 Mon Sep 17 00:00:00 2001
From: agozillon
Date: Wed, 25 Sep 2024 00:18:26 -0500
Subject: [PATCH 1/3] [Flang][Driver][Offload] Support -Xoffload-linker
argument in
https://github.com/agozillon updated
https://github.com/llvm/llvm-project/pull/109907
>From 6e29de9836f49e49558e7810ec0a8084017c22b4 Mon Sep 17 00:00:00 2001
From: agozillon
Date: Wed, 25 Sep 2024 00:18:26 -0500
Subject: [PATCH 1/4] [Flang][Driver][Offload] Support -Xoffload-linker
argument in
@@ -7326,11 +7326,13 @@ void
ItaniumMangleContextImpl::mangleCXXCtorVTable(const CXXRecordDecl *RD,
raw_ostream &Out) {
// ::= TC _
CXXNameMangler Mangler(*this, Out);
+ QualType RDType = getASTContext().getRecordType
Author: Rahman Lavaee
Date: 2024-09-25T22:03:10-07:00
New Revision: 7b7747dc1d3da1a829503ea9505b4cecce4f5bda
URL:
https://github.com/llvm/llvm-project/commit/7b7747dc1d3da1a829503ea9505b4cecce4f5bda
DIFF:
https://github.com/llvm/llvm-project/commit/7b7747dc1d3da1a829503ea9505b4cecce4f5bda.diff
https://github.com/rlavaee closed
https://github.com/llvm/llvm-project/pull/110039
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/joaosaffran updated
https://github.com/llvm/llvm-project/pull/109331
>From 50d21754119ac10c2ee2376ed8f79d12f73cd137 Mon Sep 17 00:00:00 2001
From: Joao Saffran
Date: Thu, 19 Sep 2024 00:13:51 +
Subject: [PATCH 1/5] Codegen builtin
---
clang/include/clang/Basic/Builtins.
https://github.com/joaosaffran edited
https://github.com/llvm/llvm-project/pull/109331
___
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: Chuanqi Xu (ChuanqiXu9)
Changes
According to https://github.com/ChuanqiXu9/clangd-for-modules/issues/9, I
surprisingly found the support for C++20 modules doesn't support code
completion well.
After debugging, I found there are p
https://github.com/ChuanqiXu9 created
https://github.com/llvm/llvm-project/pull/110083
According to https://github.com/ChuanqiXu9/clangd-for-modules/issues/9, I
surprisingly found the support for C++20 modules doesn't support code
completion well.
After debugging, I found there are problems:
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 2b0a708f41dd6291ee744704d43febc975e3d026
e35e600159c99736de7d2bc735c738002f592988 --e
https://github.com/fsfod updated
https://github.com/llvm/llvm-project/pull/108276
>From c842394a368fa51ddad78f9c0a7efe2026425b52 Mon Sep 17 00:00:00 2001
From: Thomas Fransham
Date: Tue, 10 Sep 2024 02:22:18 +0100
Subject: [PATCH 1/6] [Clang] Add explicit visibility symbol macros and update
CM
llvmbot wrote:
@llvm/pr-subscribers-clangd
Author: Chuanqi Xu (ChuanqiXu9)
Changes
According to https://github.com/ChuanqiXu9/clangd-for-modules/issues/9, I
surprisingly found the support for C++20 modules doesn't support code
completion well.
After debugging, I found there are problems
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-x86_64-debian-fast`
running on `gribozavr4` while building `clang,llvm` at step 6
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/56/builds/8336
Here is the
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `arc-builder` running on
`arc-worker` while building `clang,llvm` at step 5 "build-unified-tree".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/3/builds/5248
Here is the relevant piece of the build l
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-aarch64-linux-fuzzer` running on `sanitizer-buildbot12` while
building `clang,llvm` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/159/builds/6921
Here is the relevan
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `llvm-clang-x86_64-sie-win`
running on `sie-win-worker` while building `clang,llvm` at step 6
"build-unified-tree".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/46/builds/5534
Here is the relevant
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-arm-ubuntu` running
on `linaro-lldb-arm-ubuntu` while building `clang,llvm` at step 4 "build".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/18/builds/4463
Here is the relevant piece of the bui
@@ -1,7 +1,28 @@
+// Verify ubsan doesn't emit checks for ignorelisted types
+// RUN: echo "[{unsigned-integer-overflow,signed-integer-overflow}]" >
%t-int.ignorelist
+// RUN: echo "type:int" >> %t-int.ignorelist
+// RUN: %clang_cc1 -triple x86_64-linux-gnu
-fsanitize=signed-int
@@ -616,6 +626,62 @@ class LineJoiner {
return 1;
}
+ unsigned tryMergeNamespace(SmallVectorImpl::const_iterator
I,
+ SmallVectorImpl::const_iterator
E,
+ unsigned Limit) {
+if (Limit == 0)
+ return 0;
+
Author: Joseph Huber
Date: 2024-09-25T12:58:10-07:00
New Revision: eb48aac7d40ee9cd3072c466d7ab17facb58570f
URL:
https://github.com/llvm/llvm-project/commit/eb48aac7d40ee9cd3072c466d7ab17facb58570f
DIFF:
https://github.com/llvm/llvm-project/commit/eb48aac7d40ee9cd3072c466d7ab17facb58570f.diff
https://github.com/jhuber6 closed
https://github.com/llvm/llvm-project/pull/109152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rlavaee updated
https://github.com/llvm/llvm-project/pull/110039
>From 52f633c5d3597c2ece53f821ee775aebce758efb Mon Sep 17 00:00:00 2001
From: Rahman Lavaee
Date: Wed, 25 Sep 2024 20:12:07 +
Subject: [PATCH 1/2] Reapply "Deprecate the `-fbasic-block-sections=labels`
opti
https://github.com/haoNoQ approved this pull request.
Ah classic! LGTM!
https://github.com/llvm/llvm-project/pull/109393
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zygoloid wrote:
I believe all the feedback has been addressed, and plan to merge in 24 hours if
there are no further comments.
https://github.com/llvm/llvm-project/pull/109208
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
https://github.com/haoNoQ edited
https://github.com/llvm/llvm-project/pull/109393
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -102,12 +102,13 @@ class UncountedCallArgsChecker
// if ((*P)->hasAttr())
// continue;
-const auto *ArgType = (*P)->getType().getTypePtrOrNull();
-if (!ArgType)
+QualType ArgType = (*P)->getType().getCanonicalType();
+const a
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 4db0cc4c5582c73eb793572f8a8d5892b05fdfde
75cd73ecda039750270e8bde0523f51e4698c4ea --e
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Akira Hatanaka (ahatanak)
Changes
The new cast kinds are needed to distinguish between no-op conversions and
conversions from pointers to noexcept functions to pointers to functions
without noexcept as the latter can cause function pointe
llvmbot wrote:
@llvm/pr-subscribers-clangd
Author: Akira Hatanaka (ahatanak)
Changes
The new cast kinds are needed to distinguish between no-op conversions and
conversions from pointers to noexcept functions to pointers to functions
without noexcept as the latter can cause function point
https://github.com/ahatanak ready_for_review
https://github.com/llvm/llvm-project/pull/110048
___
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-clang-x86_64-sie-ubuntu-fast` running on `sie-linux-worker` while
building `clang` at step 6 "test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/144/builds/8001
Sirraide wrote:
Forgot to set the target again it seems; will be fixing that in a moment
https://github.com/llvm/llvm-project/pull/85325
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-hlsl
@llvm/pr-subscribers-clang
Author: Helena Kotas (hekota)
Changes
The `__builtin_hlsl_create_handle` called from the constructor of resource
buffer class was supposed to initialize the resource handle based on resource
type and registry binding inf
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (Sirraide)
Changes
I forgot to set the target for a test that uses the `preserve_most` cc and it’s
breaking the bots.
---
Full diff: https://github.com/llvm/llvm-project/pull/110068.diff
1 Files Affected:
- (modified) clang/test/S
https://github.com/Sirraide created
https://github.com/llvm/llvm-project/pull/110068
I forgot to set the target for a test that uses the `preserve_most` cc and it’s
breaking the bots.
>From 1b1a9b7c03383bb7fc58f62a81272dfa0ee30b83 Mon Sep 17 00:00:00 2001
From: Sirraide
Date: Thu, 26 Sep 2024
https://github.com/hekota ready_for_review
https://github.com/llvm/llvm-project/pull/109910
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Sirraide
Date: 2024-09-26T03:48:26+02:00
New Revision: 2b0a708f41dd6291ee744704d43febc975e3d026
URL:
https://github.com/llvm/llvm-project/commit/2b0a708f41dd6291ee744704d43febc975e3d026
DIFF:
https://github.com/llvm/llvm-project/commit/2b0a708f41dd6291ee744704d43febc975e3d026.diff
LOG:
https://github.com/tcwzxx edited
https://github.com/llvm/llvm-project/pull/109970
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -7326,11 +7326,13 @@ void
ItaniumMangleContextImpl::mangleCXXCtorVTable(const CXXRecordDecl *RD,
raw_ostream &Out) {
// ::= TC _
CXXNameMangler Mangler(*this, Out);
+ QualType RDType = getASTContext().getRecordType
@@ -224,6 +225,34 @@ void RISCVTargetInfo::getTargetDefines(const LangOptions
&Opts,
else
Builder.defineMacro("__riscv_32e");
}
+
+ if (Opts.CFProtectionBranch) {
+if (checkCFProtectionBranchSupported()) {
+ auto Scheme = Opts.getCFBranchLabelScheme();
+
@@ -7326,11 +7326,13 @@ void
ItaniumMangleContextImpl::mangleCXXCtorVTable(const CXXRecordDecl *RD,
raw_ostream &Out) {
// ::= TC _
CXXNameMangler Mangler(*this, Out);
+ QualType RDType = getASTContext().getRecordType
mylai-mtk wrote:
> It is useful to protect a binary with an -mcpu that doesn't support Zicfilp.
Generally I would agree that it's a good idea to protect a binary with an
`-mcpu` that doesn't support Zicfilp, but to lift the check of Zicfilp would be
misleading to naive compiler/RISC-V users w
https://github.com/hekota created
https://github.com/llvm/llvm-project/pull/110079
Resource type attributes should only ever be used on the intangible type
`__hlsl_resource_t`.
This is a draft that also includes changes from PR llvm/llvm-project#109910.
>From 970aab0a930e38dfd266c01065112602b
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 e7d68c903be0d813be96954b274e65e58c42e5e4
71c4a9361f07a606c95eb5232be50dcdff31c422 --e
kito-cheng wrote:
> @kito-cheng I need your input on this, since this would require a few updates
> on various PRs to the RISC-V specs.
Yeah, I just chat with Craig, I think that right way to go, and actually the PR
in riscv-toolchain-conventions I has implicitly allow that, I mean I only writ
https://github.com/hekota updated
https://github.com/llvm/llvm-project/pull/110079
>From 970aab0a930e38dfd266c01065112602bb274a5e Mon Sep 17 00:00:00 2001
From: Helena Kotas
Date: Wed, 25 Sep 2024 15:48:18 -0700
Subject: [PATCH 1/3] [HLSL] Allow resource type attributes only on
__hlsl_resource
mjklemm wrote:
I have seen some more matches for flang-new in the tree:
```
1:.github/workflows/release-binaries.yml:331: ninja -C ${{
steps.setup-stage.outputs.build-prefix }}/build/tools/clang/stage2-bins/ -j2
flang-new bbc
2:openmp/CMakeLists.txt:72:set(OPENMP_TEST_Fortran_COMP
Sirraide wrote:
Closing this to split it up into multiple prs.
https://github.com/llvm/llvm-project/pull/105195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide closed
https://github.com/llvm/llvm-project/pull/105195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sheredom wrote:
> > If the goal is to put virtual paths in the PCH so that you can map them
> > somewhere else in the consuming compiler's VFS, does
> > the`RedirectingFileSystem` setting `'use-external-names': false` do what
> > you need? The idea behind that setting is that we would use the
https://github.com/joaosaffran updated
https://github.com/llvm/llvm-project/pull/109331
>From 50d21754119ac10c2ee2376ed8f79d12f73cd137 Mon Sep 17 00:00:00 2001
From: Joao Saffran
Date: Thu, 19 Sep 2024 00:13:51 +
Subject: [PATCH 1/3] Codegen builtin
---
clang/include/clang/Basic/Builtins.
Author: Kazu Hirata
Date: 2024-09-25T12:34:43-07:00
New Revision: 639a0afa9955a8613902e46e168767bc05c46cdd
URL:
https://github.com/llvm/llvm-project/commit/639a0afa9955a8613902e46e168767bc05c46cdd
DIFF:
https://github.com/llvm/llvm-project/commit/639a0afa9955a8613902e46e168767bc05c46cdd.diff
L
@@ -146,13 +149,67 @@ class NoUncountedMemberChecker
BR->emitReport(std::move(Report));
haoNoQ wrote:
`setDeclWithIssue()` goes into a different PR right?
https://github.com/llvm/llvm-project/pull/108352
___
cfe-
https://github.com/haoNoQ approved this pull request.
LGTM!! I've got nitpicks but none of them are substantial enough to block.
We've figured out the ObjC thing offline right?
https://github.com/llvm/llvm-project/pull/108352
___
cfe-commits mailing l
@@ -134,10 +137,10 @@ class NoUncountedMemberChecker
Os << " in ";
printQuotedQualifiedName(Os, ClassCXXRD);
Os << " is a "
- << (isa(MemberType) ? "raw pointer" : "reference")
- << " to ref-countable type ";
+ << (isa(MemberType) ? "raw pointer" :
@@ -146,13 +149,67 @@ class NoUncountedMemberChecker
BR->emitReport(std::move(Report));
}
};
+
+class NoUncountedMemberChecker final : public RawPtrRefMemberChecker {
haoNoQ wrote:
Yes this is a perfectly valid way to reuse code here!
https://github.com
https://github.com/haoNoQ edited
https://github.com/llvm/llvm-project/pull/108352
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -53,48 +53,49 @@ hasPublicMethodInBase(const CXXBaseSpecifier *Base, const
char *NameToMatch) {
return hasPublicMethodInBaseClass(R, NameToMatch) ? R : nullptr;
}
-std::optional isRefCountable(const CXXRecordDecl* R)
-{
+std::optional isSmartPtrCompatible(const CXXRecord
@@ -0,0 +1,53 @@
+// RUN: %clang_analyze_cc1
-analyzer-checker=alpha.webkit.NoUncheckedPtrMemberChecker -verify %s
+
+#include "mock-types.h"
+#include "mock-system-header.h"
haoNoQ wrote:
Do you need to include this one everywhere? Isn't it just a tiny test to
@@ -1771,6 +1771,10 @@ def UncountedLambdaCapturesChecker :
Checker<"UncountedLambdaCapturesChecker">,
let ParentPackage = WebKitAlpha in {
+def NoUncheckedPtrMemberChecker : Checker<"NoUncheckedPtrMemberChecker">,
+ HelpText<"Check for no unchecked member variables.">,
+
@@ -4719,8 +4719,9 @@ class FunctionEffect {
NonBlocking = 1,
Sirraide wrote:
Something I only thought about just now: is `None` ever actually used?
Otherwise, we should probably just remove it and start at 0 so we don’t have to
do this `- 1` and `+ 1` dan
@@ -0,0 +1,1572 @@
+//=== SemaFunctionEffects.cpp - Sema handling of function effects
-===//
+//
+// 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: A
1 - 100 of 431 matches
Mail list logo
