@@ -478,22 +444,38 @@ static void visitLifetimeBoundArguments(IndirectLocalPath
&Path, Expr *Call,
CheckCoroObjArg = false;
if (implicitObjectParamIsLifetimeBound(Callee) || CheckCoroObjArg)
VisitLifetimeBoundArg(Callee, ObjectArg);
+else if (EnableGSLAnaly
@@ -439,6 +378,8 @@ static void visitLifetimeBoundArguments(IndirectLocalPath
&Path, Expr *Call,
if (!Callee)
return;
+ bool EnableGSLAnalysis = !Callee->getASTContext().getDiagnostics().isIgnored(
hokein wrote:
> Can we move this to VisitGSLPointerAr
https://github.com/owenca deleted
https://github.com/llvm/llvm-project/pull/101882
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -403,13 +416,25 @@ TEST_F(FormatTestComments, UnderstandsBlockComments) {
verifyFormat("f(/* aa = */\n"
" );");
- FormatStyle NoBinPacking = getLLVMStyle();
- NoBinPacking.BinPackParamet
https://github.com/hokein updated
https://github.com/llvm/llvm-project/pull/104906
>From 773a03b25a94d991206f4b517eefdf3693e6a287 Mon Sep 17 00:00:00 2001
From: Haojian Wu
Date: Tue, 20 Aug 2024 10:22:44 +0200
Subject: [PATCH 1/3] [clang] Merge the lifetimebound and GSL code paths for
lifetime
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `sanitizer-aarch64-linux`
running on `sanitizer-buildbot7` while building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/51/builds/2785
Here is the relevant piece of the b
@@ -403,13 +416,29 @@ TEST_F(FormatTestComments, UnderstandsBlockComments) {
verifyFormat("f(/* aa = */\n"
" );");
- FormatStyle NoBinPacking = getLLVMStyle();
- NoBinPacking.BinPackParamet
https://github.com/owenca deleted
https://github.com/llvm/llvm-project/pull/101882
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zyn0217 wrote:
libcxx failures are unrelated.
https://github.com/llvm/llvm-project/pull/105617
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?= ,
Jan =?utf-8?q?Kokemüller?=
Message-ID:
In-Reply-To:
@@ -6926,11 +6926,26 @@ void Sema::AddOverloadCandidate(
/// have linkage. So that all entities of the same should share one
/// linkage. But in clang, differ
https://github.com/owenca milestoned
https://github.com/llvm/llvm-project/pull/105043
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4591,7 +4589,9 @@ bool TokenAnnotator::spaceRequiredBetween(const
AnnotatedLine &Line,
if (!BeforeLeft)
return false;
if (BeforeLeft->is(tok::coloncolon)) {
- return Left.is(tok::star) &&
+ const auto *Prev = BeforeLeft->Previous;
+ return Left
https://github.com/owenca milestoned
https://github.com/llvm/llvm-project/pull/105613
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -6926,11 +6926,26 @@ void Sema::AddOverloadCandidate(
/// have linkage. So that all entities of the same should share one
/// linkage. But in clang, different entities of the same could have
/// different linkage.
-NamedDecl *ND = Function;
-if (auto *Spe
@@ -403,13 +416,29 @@ TEST_F(FormatTestComments, UnderstandsBlockComments) {
verifyFormat("f(/* aa = */\n"
" );");
- FormatStyle NoBinPacking = getLLVMStyle();
- NoBinPacking.BinPackParamet
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/101882
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -403,13 +416,25 @@ TEST_F(FormatTestComments, UnderstandsBlockComments) {
verifyFormat("f(/* aa = */\n"
" );");
- FormatStyle NoBinPacking = getLLVMStyle();
- NoBinPacking.BinPackParamet
cor3ntin wrote:
I would recommend creating an issue and see how ewg handles the new information
before reaping out a lot of code.
https://github.com/llvm/llvm-project/pull/101448
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.ll
https://github.com/EthanLuisMcDonough closed
https://github.com/llvm/llvm-project/pull/102691
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ethan Luis McDonough
Date: 2024-08-22T01:10:54-05:00
New Revision: fde2d23ee2a204050a210f2f7b290643a272f737
URL:
https://github.com/llvm/llvm-project/commit/fde2d23ee2a204050a210f2f7b290643a272f737
DIFF:
https://github.com/llvm/llvm-project/commit/fde2d23ee2a204050a210f2f7b290643a272f73
@@ -4387,12 +4387,12 @@ Simple Constants
zeros. So '``s0x0001``' of type '``i16``' will be -1, not 1.
**Floating-point constants**
Floating-point constants use standard decimal notation (e.g.
-123.421), exponential notation (e.g. 1.23421e+2), or a more precise
-
@@ -4426,6 +4425,32 @@ represented by ``0xH`` followed by 4 hexadecimal digits.
The bfloat 16-bit
format is represented by ``0xR`` followed by 4 hexadecimal digits. All
hexadecimal formats are big-endian (sign bit at the left).
+Some of the special floating point values can b
@@ -4426,6 +4425,32 @@ represented by ``0xH`` followed by 4 hexadecimal digits.
The bfloat 16-bit
format is represented by ``0xR`` followed by 4 hexadecimal digits. All
hexadecimal formats are big-endian (sign bit at the left).
+Some of the special floating point values can b
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-llvm-globalisel
Author: Min-Yih Hsu (mshockwave)
Changes
Instead of hexidecimal values, print `nan`, `pinf`, and `ninf` when the
floating point constant is equal to positive quiet NaN with zero payload,
positive infinity, and
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/105617
___
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: Younan Zhang (zyn0217)
Changes
Follow-up on 8ac140f39.
The test `SemaTemplate/default-parm-init.cpp` was introduced since the fix for
#80288 and mainly did the following things:
- Ensure the default arguments are properly substituted ins
llvmbot wrote:
@llvm/pr-subscribers-libunwind
Author: Ian Anderson (ian-twilightcoder)
Changes
libunwind shouldn't know that compact_unwind_encoding.h is part of a MachO
module that it doesn't own. Delete the mach-o module map, and let whatever is
in charge of the mach-o directory be the
https://github.com/zyn0217 created
https://github.com/llvm/llvm-project/pull/105617
Follow-up on 8ac140f39.
The test `SemaTemplate/default-parm-init.cpp` was introduced since the fix for
#80288 and mainly did the following things:
- Ensure the default arguments are properly substituted inside
https://github.com/ChuanqiXu9 closed
https://github.com/llvm/llvm-project/pull/105610
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: h-vetinari
Date: 2024-08-22T13:32:40+08:00
New Revision: ded6dd244cce3e683201a668ce321d4474baa8fb
URL:
https://github.com/llvm/llvm-project/commit/ded6dd244cce3e683201a668ce321d4474baa8fb
DIFF:
https://github.com/llvm/llvm-project/commit/ded6dd244cce3e683201a668ce321d4474baa8fb.diff
LO
https://github.com/ChuanqiXu9 approved this pull request.
Oh, I forgot it. Thanks~
https://github.com/llvm/llvm-project/pull/105610
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ian-twilightcoder created
https://github.com/llvm/llvm-project/pull/105616
libunwind shouldn't know that compact_unwind_encoding.h is part of a MachO
module that it doesn't own. Delete the mach-o module map, and let whatever is
in charge of the mach-o directory be the one to
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Owen Pan (owenca)
Changes
Fixes #105480.
---
Full diff: https://github.com/llvm/llvm-project/pull/105613.diff
2 Files Affected:
- (modified) clang/lib/Format/TokenAnnotator.cpp (+2-2)
- (modified) clang/unittests/Format/TokenAnn
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/105613
Fixes #105480.
>From 4bb6856deb83b89a5530efca2155da40663753e1 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Wed, 21 Aug 2024 21:45:12 -0700
Subject: [PATCH] [clang-format] Correctly compute SplitPenalty of
Trai
https://github.com/EthanLuisMcDonough updated
https://github.com/llvm/llvm-project/pull/102691
>From 24b1a99a1c014e1015fbba137430c5c6f3e414c5 Mon Sep 17 00:00:00 2001
From: Ethan Luis McDonough
Date: Fri, 28 Jun 2024 12:39:19 -0500
Subject: [PATCH 1/6] Changes from old gpuprof branch
---
clan
@@ -2854,10 +2854,121 @@ void CodeGenFunction::EmitMultiVersionResolver(
case llvm::Triple::aarch64:
EmitAArch64MultiVersionResolver(Resolver, Options);
return;
+ case llvm::Triple::riscv32:
+ case llvm::Triple::riscv64:
+EmitRISCVMultiVersionResolver(Resolver,
https://github.com/ergawy updated
https://github.com/llvm/llvm-project/pull/104664
>From 205ed1497a145664471055b24ea0391f93b30711 Mon Sep 17 00:00:00 2001
From: ergawy
Date: Sat, 17 Aug 2024 00:20:11 -0500
Subject: [PATCH] [flang][driver] Add pre-processing type to `.i` files
This diff allows
@@ -4368,8 +4363,19 @@ template bool
Compiler::visitIfStmt(const IfStmt *IS) {
if (!visitDeclStmt(CondDecl))
return false;
- if (!this->visitBool(IS->getCond()))
-return false;
+ // Compile condition.
+ if (IS->isNonNegatedConsteval()) {
+if (!this->emit
ergawy wrote:
> Please bear with me, it's been a while since I've touched this and also, I am
> afk 😅
No worries. I am new to this part of the code and might be misinterpreting
things myself.
> IIRC, the generated temp files (e.g. *.i) are used for the subsequent
> compilation phases. So, af
https://github.com/matinraayai ready_for_review
https://github.com/llvm/llvm-project/pull/105541
___
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: None (h-vetinari)
Changes
This landed as https://github.com/llvm/llvm-project/pull/102287 for main &
https://github.com/llvm/llvm-project/pull/102561 for 19.x
CC @ChuanqiXu9
---
Full diff: https://github.com/llvm/llvm-project/pu
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (h-vetinari)
Changes
This landed as https://github.com/llvm/llvm-project/pull/102287 for main &
https://github.com/llvm/llvm-project/pull/102561 for 19.x
CC @ChuanqiXu9
---
Full diff: https://github.com/llvm/llvm-project/pull/10561
https://github.com/h-vetinari created
https://github.com/llvm/llvm-project/pull/105610
This landed as https://github.com/llvm/llvm-project/pull/102287 for main &
https://github.com/llvm/llvm-project/pull/102561 for 19.x
CC @ChuanqiXu9
>From 866b4834ec847c8459defd1b63b5d227d03e2761 Mon Sep 17
https://github.com/v01dXYZ edited
https://github.com/llvm/llvm-project/pull/105608
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/EthanLuisMcDonough updated
https://github.com/llvm/llvm-project/pull/102691
>From 24b1a99a1c014e1015fbba137430c5c6f3e414c5 Mon Sep 17 00:00:00 2001
From: Ethan Luis McDonough
Date: Fri, 28 Jun 2024 12:39:19 -0500
Subject: [PATCH 1/6] Changes from old gpuprof branch
---
clan
koachan wrote:
> We have also developed some performance related patches for the Sparc
> backend, but it has been difficult to get them upstreamed as, as you say, few
> people are working on the Sparc backend.
This is off-topic but is it possible for you to submit it (or if I can see
those)?
https://github.com/v01dXYZ updated
https://github.com/llvm/llvm-project/pull/105608
>From faef073dab6d45f1b50fc877e5cf336a3427ecfa Mon Sep 17 00:00:00 2001
From: v01dxyz
Date: Thu, 22 Aug 2024 03:55:27 +0200
Subject: [PATCH] [Clang][AArch64] Fix typo with colon-separated syntax for
system regi
llvmbot wrote:
@llvm/pr-subscribers-backend-aarch64
@llvm/pr-subscribers-backend-arm
Author: None (v01dXYZ)
Changes
The range for Op0 was set to 1 instead of 3.
The description of e493f17784a9c6000ca7c92499233490f1d1 visually explains
the encoding of implementation-defined system reg
https://github.com/v01dXYZ created
https://github.com/llvm/llvm-project/pull/105608
The range for Op0 was set to 1 instead of 3.
The description of e493f17784a9c6000ca7c92499233490f1d1 visually explains
the encoding of implementation-defined system registers.
Gobolt: https://godbolt.org/z
https://github.com/farzonl edited
https://github.com/llvm/llvm-project/pull/104856
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/farzonl approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/104856
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,53 @@
+//===-
DirectX.cpp-===//
+//
+// 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/topperc closed
https://github.com/llvm/llvm-project/pull/105148
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Craig Topper
Date: 2024-08-21T19:23:07-07:00
New Revision: 0ca77f6656a772624a591261957f6b313a0d544e
URL:
https://github.com/llvm/llvm-project/commit/0ca77f6656a772624a591261957f6b313a0d544e
DIFF:
https://github.com/llvm/llvm-project/commit/0ca77f6656a772624a591261957f6b313a0d544e.diff
@@ -4591,7 +4589,9 @@ bool TokenAnnotator::spaceRequiredBetween(const
AnnotatedLine &Line,
if (!BeforeLeft)
return false;
if (BeforeLeft->is(tok::coloncolon)) {
- return Left.is(tok::star) &&
+ const auto *Prev = BeforeLeft->Previous;
+ return Left
@@ -4591,7 +4589,9 @@ bool TokenAnnotator::spaceRequiredBetween(const
AnnotatedLine &Line,
if (!BeforeLeft)
return false;
if (BeforeLeft->is(tok::coloncolon)) {
- return Left.is(tok::star) &&
+ const auto *Prev = BeforeLeft->Previous;
+ return Left
https://github.com/owenca updated
https://github.com/llvm/llvm-project/pull/105043
>From 4aa47d190a84ecd0432dc9b6db1d38b296f4df23 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Tue, 20 Aug 2024 06:44:41 -0700
Subject: [PATCH 1/2] [clang-format] Don't insert a space between :: and *
Also, don't
@@ -0,0 +1,131 @@
+// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.3-library -x hlsl -o -
-fsyntax-only %s -verify
+
+// TODO: Implement "Buffer", we use a substitute UDT
+// to test the 't' binding type for this test.
+
+template
+struct [[hlsl::resource_class(SRV)]] MyTemplated
https://github.com/damyanp requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/97103
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -459,7 +467,408 @@ void SemaHLSL::handleResourceClassAttr(Decl *D, const
ParsedAttr &AL) {
D->addAttr(HLSLResourceClassAttr::Create(getASTContext(), RC, ArgLoc));
}
-void SemaHLSL::handleResourceBindingAttr(Decl *D, const ParsedAttr &AL) {
+struct RegisterBindingFlags {
@@ -0,0 +1,71 @@
+// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.3-library -x hlsl -o -
-fsyntax-only %s -verify
+
+// This test validates the diagnostics that are emitted when a variable with a
"resource" type
+// is bound to a register using the register annotation
+
+/*
+tem
@@ -31,6 +31,7 @@ class SemaHLSL : public SemaBase {
public:
SemaHLSL(Sema &S);
+ HLSLResourceAttr *mergeHLSLResourceAttr(bool CBuffer);
damyanp wrote:
I don't think `mergeHLSLResourceAttr` exists any more?
https://github.com/llvm/llvm-project/pull/97103
https://github.com/giulianobelinassi updated
https://github.com/llvm/llvm-project/pull/105591
>From 3a1a241f925e7b6dcc6c8701ab83b5fb04f18853 Mon Sep 17 00:00:00 2001
From: Giuliano Belinassi
Date: Wed, 21 Aug 2024 18:31:36 -0300
Subject: [PATCH] Fix the behavior of __COUNT__ macros when PCH is
https://github.com/eddyz87 closed
https://github.com/llvm/llvm-project/pull/105417
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: eddyz87
Date: 2024-08-22T03:40:56+03:00
New Revision: 64e464349bfca0d90e07f6db2f710d4d53cdacd4
URL:
https://github.com/llvm/llvm-project/commit/64e464349bfca0d90e07f6db2f710d4d53cdacd4
DIFF:
https://github.com/llvm/llvm-project/commit/64e464349bfca0d90e07f6db2f710d4d53cdacd4.diff
LOG:
https://github.com/brendandahl updated
https://github.com/llvm/llvm-project/pull/105434
>From e992578b7269c365e619fe201e7cc703149c7067 Mon Sep 17 00:00:00 2001
From: Brendan Dahl
Date: Tue, 20 Aug 2024 21:55:47 +
Subject: [PATCH] [WebAssembly] Change half-precision feature name to fp16.
Th
https://github.com/tlively approved this pull request.
https://github.com/llvm/llvm-project/pull/105434
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bob80905 updated
https://github.com/llvm/llvm-project/pull/104239
>From aaa455933d3703b84634703fd4fcb5c815aa139e Mon Sep 17 00:00:00 2001
From: Joshua Batista
Date: Wed, 14 Aug 2024 14:02:22 -0700
Subject: [PATCH 1/4] create texture dimension attr
---
clang/include/clang/Ba
https://github.com/galenelias updated
https://github.com/llvm/llvm-project/pull/105597
>From 4118b7dde9adbee7b6aaf5d094d34cb6b64f6c77 Mon Sep 17 00:00:00 2001
From: Galen Elias
Date: Wed, 21 Aug 2024 16:33:42 -0700
Subject: [PATCH] clang-format: Add "AllowShortNamespacesOnASingleLine" option
T
https://github.com/galenelias updated
https://github.com/llvm/llvm-project/pull/105597
>From 3819c15c8c6258d1d29a38fcfb71db8567fb34e5 Mon Sep 17 00:00:00 2001
From: Galen Elias
Date: Wed, 21 Aug 2024 15:46:35 -0700
Subject: [PATCH] clang-format: Add "AllowShortNamespacesOnASingleLine" option
T
Sirraide wrote:
> CWG looked at 2917 last week, and changed the status of the issue. Can you
> revisit your test for it?
Ah, something that was previously accepted is now disallowed; I’ll implement
that tomorrow.
https://github.com/llvm/llvm-project/pull/101448
___
Endilll wrote:
@Sirraide CWG looked at 2917 last week, and changed the status of the issue.
Can you revisit your test for it?
https://cplusplus.github.io/CWG/issues/2917.html
https://github.com/llvm/llvm-project/pull/101448
___
cfe-commits mailing li
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Galen Elias (galenelias)
Changes
This addresses: https://github.com/llvm/llvm-project/issues/101363 which is a
resurrection of a previously opened but never completed review:
https://reviews.llvm.org/D11851
The feature is to allow
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it
https://github.com/galenelias created
https://github.com/llvm/llvm-project/pull/105597
This addresses: https://github.com/llvm/llvm-project/issues/101363 which is a
resurrection of a previously opened but never completed review:
https://reviews.llvm.org/D11851
The feature is to allow code lik
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/105544
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,101 @@
+// RUN: %clang_cc1 -std=c++20 -Wno-all -Wunsafe-buffer-usage \
+// RUN:-verify %s
+
+typedef struct {} FILE;
+void memcpy();
+void __asan_memcpy();
+void strcpy();
+void strcpy_s();
+void wcscpy_s();
+unsigned strlen( const char* str );
+int fprintf(
@@ -0,0 +1,101 @@
+// RUN: %clang_cc1 -std=c++20 -Wno-all -Wunsafe-buffer-usage \
+// RUN:-verify %s
+
+typedef struct {} FILE;
+void memcpy();
+void __asan_memcpy();
+void strcpy();
+void strcpy_s();
+void wcscpy_s();
+unsigned strlen( const char* str );
+int fprintf(
@@ -0,0 +1,54 @@
+//===- DirectX.cpp
+//---===//
hekota wrote:
Oops ;)
https://github.com/llvm/llvm-project/pull/104856
___
cfe-commits mailing list
cfe-commits@l
https://github.com/hekota updated
https://github.com/llvm/llvm-project/pull/104856
>From 44e814b925a1ad8ac40fe6904542cbade516c065 Mon Sep 17 00:00:00 2001
From: Helena Kotas
Date: Mon, 19 Aug 2024 13:34:13 -0700
Subject: [PATCH 1/3] [DirectX] Add DirectXTargetCodeGenInfo
Adds TargetCodeGenInfo
https://github.com/giulianobelinassi updated
https://github.com/llvm/llvm-project/pull/105591
>From 914427af9b83848ac395bbff663133aadf387161 Mon Sep 17 00:00:00 2001
From: Giuliano Belinassi
Date: Wed, 21 Aug 2024 18:31:36 -0300
Subject: [PATCH] Fix the behavior of __COUNT__ macros when PCH is
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 ab86fc74c04ff508f909b7b6131df1551dd833fc
259653e147aff62c70dd2c16d46886ae2c92412d --e
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
@llvm/pr-subscribers-clang
Author: Giuliano Belinassi (giulianobelinassi)
Changes
Previoulsy, calling `ASTUnit::LoadFromCompilerInvocation` with
`PrecompilePreambleAfterNParses > 0` caused the `__COUNT__` macro value to
be restarted. Thi
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it
https://github.com/giulianobelinassi created
https://github.com/llvm/llvm-project/pull/105591
Previoulsy, calling `ASTUnit::LoadFromCompilerInvocation` with
`PrecompilePreambleAfterNParses > 0` caused the `__COUNT__` macro value to be
restarted. This commit fixes this by remembering the value
@@ -443,6 +448,368 @@ AST_MATCHER(ArraySubscriptExpr, isSafeArraySubscript) {
return false;
}
+namespace libc_fun_disjoint_inner_matchers {
+// `libc_fun_disjoint_inner_matchers` covers a set of matchers that match
+// disjoint node sets. They all take a `CoreName`, which
@@ -0,0 +1,101 @@
+// RUN: %clang_cc1 -std=c++20 -Wno-all -Wunsafe-buffer-usage \
+// RUN:-verify %s
+
+typedef struct {} FILE;
+void memcpy();
+void __asan_memcpy();
+void strcpy();
+void strcpy_s();
+void wcscpy_s();
+unsigned strlen( const char* str );
+int fprintf(
@@ -443,6 +449,396 @@ AST_MATCHER(ArraySubscriptExpr, isSafeArraySubscript) {
return false;
}
+AST_MATCHER_P(CallExpr, hasNumArgs, unsigned, Num) {
+ return Node.getNumArgs() == Num;
+}
+
+namespace libc_func_matchers {
+// Under `libc_func_matchers`, define a set of matche
https://github.com/haoNoQ commented:
Mostly LGTM! I don't have major concerns.
https://github.com/llvm/llvm-project/pull/101583
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -12383,6 +12383,13 @@ def warn_unsafe_buffer_operation : Warning<
"%select{unsafe pointer operation|unsafe pointer arithmetic|"
"unsafe buffer access|function introduces unsafe buffer manipulation|unsafe
invocation of span::data}0">,
InGroup, DefaultIgnore;
+def warn_
@@ -12383,6 +12383,13 @@ def warn_unsafe_buffer_operation : Warning<
"%select{unsafe pointer operation|unsafe pointer arithmetic|"
"unsafe buffer access|function introduces unsafe buffer manipulation|unsafe
invocation of span::data}0">,
InGroup, DefaultIgnore;
+def warn_
@@ -1025,6 +1421,92 @@ class DataInvocationGadget : public WarningGadget {
DeclUseList getClaimedVarUseSites() const override { return {}; }
};
+class UnsafeLibcFunctionCallGadget : public WarningGadget {
+ const CallExpr *const Call;
+ constexpr static const char *const T
@@ -443,6 +449,396 @@ AST_MATCHER(ArraySubscriptExpr, isSafeArraySubscript) {
return false;
}
+AST_MATCHER_P(CallExpr, hasNumArgs, unsigned, Num) {
+ return Node.getNumArgs() == Num;
+}
+
+namespace libc_func_matchers {
+// Under `libc_func_matchers`, define a set of matche
https://github.com/haoNoQ edited
https://github.com/llvm/llvm-project/pull/101583
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/topperc updated
https://github.com/llvm/llvm-project/pull/105148
>From b7a0e87790b1d17201728dba1a9c2149fc7d9a17 Mon Sep 17 00:00:00 2001
From: Craig Topper
Date: Tue, 20 Aug 2024 10:13:56 -0700
Subject: [PATCH 1/3] [RISCV] Add CSRs and a instruction for Smctr and Ssctr
exten
Author: Dmitri Gribenko
Date: 2024-08-21T23:49:45+02:00
New Revision: f709cd5add0ea36bb14259e9716bd74e5c762128
URL:
https://github.com/llvm/llvm-project/commit/f709cd5add0ea36bb14259e9716bd74e5c762128
DIFF:
https://github.com/llvm/llvm-project/commit/f709cd5add0ea36bb14259e9716bd74e5c762128.dif
hazzlim wrote:
> > adding nuw is also valid for signed indices here
>
> I don't understand how you think this would work; a-1 and a+-1 are required
> to produce the same result.
The thinking here RE signed indices was that in this special case, where the
base address of the GEP is the start o
https://github.com/efriedma-quic approved this pull request.
LGTM; the approach here seems appropriate.
(The EmitCheckedInBoundsGEP API could probably use a refactor so we don't have
calls to isSignedOverflowDefined() scattered all over the place, but that's a
separate issue.)
https://github.
cor3ntin wrote:
@ldionne and @philnik777 you have an opinion on that builtin?
https://github.com/llvm/llvm-project/pull/91895
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2887,6 +2887,10 @@ def fxray_selected_function_group :
HelpText<"When using -fxray-function-groups, select which group of functions
to instrument. Valid range is 0 to fxray-function-groups - 1">,
MarshallingInfoInt, "0">;
+def fxray_enable_shared : Flag<["-"], "fxray-
1 - 100 of 328 matches
Mail list logo