https://github.com/philnik777 converted_to_draft
https://github.com/llvm/llvm-project/pull/81213
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Owen Pan
Date: 2024-02-12T00:04:34-08:00
New Revision: f5d71b79fe978f1e70f4a41adfd15f65dec6e210
URL:
https://github.com/llvm/llvm-project/commit/f5d71b79fe978f1e70f4a41adfd15f65dec6e210
DIFF:
https://github.com/llvm/llvm-project/commit/f5d71b79fe978f1e70f4a41adfd15f65dec6e210.diff
LOG:
Author: Owen Pan
Date: 2024-02-12T00:07:47-08:00
New Revision: 33108fae9020cb49577fc3ee0aed219cc581f82a
URL:
https://github.com/llvm/llvm-project/commit/33108fae9020cb49577fc3ee0aed219cc581f82a
DIFF:
https://github.com/llvm/llvm-project/commit/33108fae9020cb49577fc3ee0aed219cc581f82a.diff
LOG:
Author: Owen Pan
Date: 2024-02-12T00:10:24-08:00
New Revision: 7f40c5cc4e15a0a67b031f13370afc342a5dc14b
URL:
https://github.com/llvm/llvm-project/commit/7f40c5cc4e15a0a67b031f13370afc342a5dc14b
DIFF:
https://github.com/llvm/llvm-project/commit/7f40c5cc4e15a0a67b031f13370afc342a5dc14b.diff
LOG:
https://github.com/zyn0217 created
https://github.com/llvm/llvm-project/pull/81449
This fixes a regression since
https://github.com/llvm/llvm-project/commit/340eac01f7dad6c24cee35dd35f2484098dd6b1a,
from which we compared function parameter types with cv-qualifiers taken into
account. However
https://github.com/sdesmalen-arm approved this pull request.
LGTM with nit addressed
https://github.com/llvm/llvm-project/pull/81268
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sdesmalen-arm edited
https://github.com/llvm/llvm-project/pull/81268
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3501,9 +3501,16 @@ bool Sema::checkTargetAttr(SourceLocation LiteralLoc,
StringRef AttrStr) {
return false;
}
+static bool hasStreamingModeChangeInABI(const FunctionDecl *FD) {
sdesmalen-arm wrote:
nit:
```suggestion
static bool hasArmStreamingInterfac
https://github.com/zyn0217 updated
https://github.com/llvm/llvm-project/pull/81449
>From f4fb72075b8b3242656c10618eb5baa6a4204907 Mon Sep 17 00:00:00 2001
From: Younan Zhang
Date: Mon, 12 Feb 2024 15:47:59 +0800
Subject: [PATCH 1/2] [Clang][Sema] Don't consider top-level cv-qualifiers in
templ
https://github.com/zyn0217 updated
https://github.com/llvm/llvm-project/pull/81449
>From f4fb72075b8b3242656c10618eb5baa6a4204907 Mon Sep 17 00:00:00 2001
From: Younan Zhang
Date: Mon, 12 Feb 2024 15:47:59 +0800
Subject: [PATCH 1/2] [Clang][Sema] Don't consider top-level cv-qualifiers in
templ
https://github.com/chapuni updated
https://github.com/llvm/llvm-project/pull/81227
>From c2b49a5317bf5b8af419cba814f95cc9305bec21 Mon Sep 17 00:00:00 2001
From: NAKAMURA Takumi
Date: Thu, 8 Feb 2024 23:12:54 +0900
Subject: [PATCH 1/3] [MC/DC] Refactor: Make `MCDCParams` as `std::variant`
Intro
zyn0217 wrote:
The Windows CI is still not working; I ran the libc++ tests locally and they
are green.
https://github.com/llvm/llvm-project/pull/81449
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
https://github.com/zyn0217 ready_for_review
https://github.com/llvm/llvm-project/pull/81449
___
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
This fixes a regression since
https://github.com/llvm/llvm-project/commit/340eac01f7dad6c24cee35dd35f2484098dd6b1a,
from which we compared function parameter types with cv-qualifiers taken into
account. How
https://github.com/Pierre-vh closed
https://github.com/llvm/llvm-project/pull/76955
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Mariya Podchishchaeva
Date: 2024-02-12T10:44:20+01:00
New Revision: c90114c993ec1443360769a4d51963e3536de0ef
URL:
https://github.com/llvm/llvm-project/commit/c90114c993ec1443360769a4d51963e3536de0ef
DIFF:
https://github.com/llvm/llvm-project/commit/c90114c993ec1443360769a4d51963e3536de0
https://github.com/Fznamznon closed
https://github.com/llvm/llvm-project/pull/74512
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chapuni created
https://github.com/llvm/llvm-project/pull/81459
They can be also used in `clang`.
Introduce the lightweight header instead of `CoverageMapping.h`.
>From 03bab9acf5aa3bf13c2223e92dbad040202d34bd Mon Sep 17 00:00:00 2001
From: NAKAMURA Takumi
Date: Mon, 12 Feb
llvmbot wrote:
@llvm/pr-subscribers-pgo
Author: NAKAMURA Takumi (chapuni)
Changes
They can be also used in `clang`.
Introduce the lightweight header instead of `CoverageMapping.h`.
---
Full diff: https://github.com/llvm/llvm-project/pull/81459.diff
6 Files Affected:
- (modified) clang/
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: NAKAMURA Takumi (chapuni)
Changes
They can be also used in `clang`.
Introduce the lightweight header instead of `CoverageMapping.h`.
---
Full diff: https://github.com/llvm/llvm-project/pull/81459.diff
6 Files Affected:
- (modified) clan
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: NAKAMURA Takumi (chapuni)
Changes
They can be also used in `clang`.
Introduce the lightweight header instead of `CoverageMapping.h`.
---
Full diff: https://github.com/llvm/llvm-project/pull/81459.diff
6 Files Affected:
- (modifi
https://github.com/jwestwood921 closed
https://github.com/llvm/llvm-project/pull/81301
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/domin144 approved this pull request.
https://github.com/llvm/llvm-project/pull/75031
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chapuni updated
https://github.com/llvm/llvm-project/pull/81459
>From 03bab9acf5aa3bf13c2223e92dbad040202d34bd Mon Sep 17 00:00:00 2001
From: NAKAMURA Takumi
Date: Mon, 12 Feb 2024 18:43:56 +0900
Subject: [PATCH 1/2] [MC/DC] Refactor: Introduce `MCDCTypes.h` for the
namespac
RKSimon wrote:
Are we assuming any particular relationship to __builtin_readcyclecounter in
terms of scales etc?
__builtin_readsteadycounter could be used to access x86 MPERF clock counters,
but to access the corresponding APERF clock we'd then need a
__builtin_readvariablecounter equivalent
https://github.com/balazske created
https://github.com/llvm/llvm-project/pull/81461
This is only a code reformatting and rename of variables to the newer format.
From d1078be3394fc240de014f72116e18d6740b5773 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bal=C3=A1zs=20K=C3=A9ri?=
Date: Mon, 12 Feb 2
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Balázs Kéri (balazske)
Changes
This is only a code reformatting and rename of variables to the newer format.
---
Full diff: https://github.com/llvm/llvm-project/pull/81461.diff
1 Files Affected:
- (modified) clang/lib/StaticAnalyzer/Che
ostannard wrote:
Ping, the ABI specification change
(https://github.com/ARM-software/abi-aa/pull/232) has now been merged.
https://github.com/llvm/llvm-project/pull/74460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
https://github.com/usx95 edited https://github.com/llvm/llvm-project/pull/80698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MDevereau updated
https://github.com/llvm/llvm-project/pull/78169
>From aa9b6e584e5f469d5ed5d8ceaa56c3b424a98b02 Mon Sep 17 00:00:00 2001
From: Matt Devereau
Date: Thu, 11 Jan 2024 17:21:10 +
Subject: [PATCH] [AArch64][SME2] Refactor arm_sme.td into multiclasses
Create s
https://github.com/MDevereau updated
https://github.com/llvm/llvm-project/pull/78169
>From 79bf1ffc720d97e96bda477b445502d2cbe71369 Mon Sep 17 00:00:00 2001
From: Matt Devereau
Date: Thu, 11 Jan 2024 17:21:10 +
Subject: [PATCH] [AArch64][SME2] Refactor arm_sme.td into multiclasses
Create s
https://github.com/jwestwood921 created
https://github.com/llvm/llvm-project/pull/81466
Added functionality to the ARMTargetParser to obtain the minor version of each
architecture, and amended ARM.cpp and AArch64.cpp to define __ARM_ARCH
correctly.
In ACLE 5.4.1, __ARM_ARCH is defined as equa
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
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: James Westwood (jwestwood921)
Changes
Added functionality to the ARMTargetParser to obtain the minor version of each
architecture, and amended ARM.cpp and AArch64.cpp to define __ARM_ARCH
correctly.
In ACLE 5.4.1, __ARM_ARCH is defined a
llvmbot wrote:
@llvm/pr-subscribers-backend-aarch64
Author: James Westwood (jwestwood921)
Changes
Added functionality to the ARMTargetParser to obtain the minor version of each
architecture, and amended ARM.cpp and AArch64.cpp to define __ARM_ARCH
correctly.
In ACLE 5.4.1, __ARM_ARCH is
github-actions[bot] wrote:
⚠️ We detected that you are using a GitHub private e-mail address to contribute
to the repo.
Please turn off [Keep my email addresses
private](https://github.com/settings/emails) setting in your account.
See [LLVM
Discourse](https://discourse.llvm.org/t/hidden-em
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 c609846155279090a4f9e659f63fb851e4946cb7
4b91b1be657dcf01d5c7f70d4c62a8ba60185efd --
https://github.com/usx95 updated https://github.com/llvm/llvm-project/pull/80698
>From 28d90711ff8e4924135d4bd4e5f252d96ac41b93 Mon Sep 17 00:00:00 2001
From: Utkarsh Saxena
Date: Tue, 23 Jan 2024 20:18:25 +
Subject: [PATCH 01/11] [misc-coroutine-hostile-raii] Use getOperand instead of
getC
https://github.com/jwestwood921 updated
https://github.com/llvm/llvm-project/pull/81466
>From 08628a75872266bdb1365ed39754512b1a400a70 Mon Sep 17 00:00:00 2001
From: James Westwood
Date: Fri, 9 Feb 2024 19:06:20 +
Subject: [PATCH 1/2] __ARM_ARCH macro definition fix
Added functionality to
vinayakdsci wrote:
@Endilll sorry for repeated pings, but I think @cor3ntin might be possibly busy.
If this looks good, can we go ahead with this PR?
Thanks!
https://github.com/llvm/llvm-project/pull/81127
___
cfe-commits mailing list
cfe-commits@lis
cor3ntin wrote:
We have a policy against more than one ping per week, thanks.
FYI, if we are not going to fix the C++11 case, we should be very explicit
about it. IE this PR is not a complete fix of
https://github.com/llvm/llvm-project/issues/79518
https://github.com/llvm/llvm-project/pull/81
dtcxzyw wrote:
Can we implement this in `computeKnownBitsFromOperator/getRangeForIntrinsic`?
https://github.com/llvm/llvm-project/blob/b21e3282864c9f7ad656c64bc375f5869ef76d19/llvm/lib/Analysis/ValueTracking.cpp#L1578-L1584
https://github.com/llvm/llvm-project/pull/79975
https://github.com/usx95 edited https://github.com/llvm/llvm-project/pull/80698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/usx95 edited https://github.com/llvm/llvm-project/pull/80698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Bäder
Date: 2024-02-12T13:27:42+01:00
New Revision: 268799cc59ef7af237278d058c6e2ee2c8eb227d
URL:
https://github.com/llvm/llvm-project/commit/268799cc59ef7af237278d058c6e2ee2c8eb227d
DIFF:
https://github.com/llvm/llvm-project/commit/268799cc59ef7af237278d058c6e2ee2c8eb227d.diff
LO
vinayakdsci wrote:
Sorry about the pings, I think I got a little ahead of myself there.
But for C++11, we got to the conclusion that copy elision gives different
behaviour for C++17 and later and thus different diagnostics?
In case I made a mistake, I'd be very happy to fix it.
Thanks
P.S. Sorr
https://github.com/arsenm edited https://github.com/llvm/llvm-project/pull/79035
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm commented:
Needs documentation in AMDGPUUsage. Should also clarify behavior of 0
https://github.com/llvm/llvm-project/pull/79035
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/l
@@ -356,6 +356,19 @@ void AMDGPUTargetCodeGenInfo::setFunctionDeclAttributes(
if (NumVGPR != 0)
F->addFnAttr("amdgpu-num-vgpr", llvm::utostr(NumVGPR));
}
+
+ if (const auto *Attr = FD->getAttr()) {
+uint32_t X = Attr->getNumWorkGroupsX();
+uint32_t Y = Attr
@@ -1108,3 +1108,8 @@ void
GCNUserSGPRUsageInfo::allocKernargPreloadSGPRs(unsigned NumSGPRs) {
unsigned GCNUserSGPRUsageInfo::getNumFreeUserSGPRs() {
return AMDGPU::getMaxNumUserSGPRs(ST) - NumUsedUserSGPRs;
}
+
+SmallVector
arsenm wrote:
std::array<3>?
ht
@@ -0,0 +1,84 @@
+; RUN: llc -mtriple=amdgcn-amd-amdhsa -mcpu=gfx900 -verify-machineinstrs < %s
| FileCheck %s
arsenm wrote:
Don't need -verify-machineinstrs
https://github.com/llvm/llvm-project/pull/79035
___
cfe-co
@@ -356,6 +356,19 @@ void AMDGPUTargetCodeGenInfo::setFunctionDeclAttributes(
if (NumVGPR != 0)
F->addFnAttr("amdgpu-num-vgpr", llvm::utostr(NumVGPR));
}
+
+ if (const auto *Attr = FD->getAttr()) {
+uint32_t X = Attr->getNumWorkGroupsX();
+uint32_t Y = Attr
@@ -2031,6 +2031,13 @@ def AMDGPUNumVGPR : InheritableAttr {
let Subjects = SubjectList<[Function], ErrorDiag, "kernel functions">;
}
+def AMDGPUNumWorkGroups : InheritableAttr {
+ let Spellings = [Clang<"amdgpu_num_work_groups", 0>];
arsenm wrote:
max_num
arsenm wrote:
Also should have follow up patch to propagate in AMDGPUAttributor, and another
to lower to !range in AMDGPULowerKernelAttributes
https://github.com/llvm/llvm-project/pull/79035
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
https://github.com/usx95 updated https://github.com/llvm/llvm-project/pull/80698
>From 28d90711ff8e4924135d4bd4e5f252d96ac41b93 Mon Sep 17 00:00:00 2001
From: Utkarsh Saxena
Date: Tue, 23 Jan 2024 20:18:25 +
Subject: [PATCH 01/12] [misc-coroutine-hostile-raii] Use getOperand instead of
getC
martinboehme wrote:
CI failure appears to be an infrastructure failure
https://github.com/llvm/llvm-project/pull/81239
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/domin144 created
https://github.com/llvm/llvm-project/pull/81474
The deduced "-march=" option always started with aarch64, which is not a valid
value. There was also no way to distinguish between armv8-r and armv8-a. After
this commit, the deduced "-march=" option will start
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Dominik Wójt (domin144)
Changes
The deduced "-march=" option always started with aarch64, which is not a valid
value. There was also no way to distinguish between armv8-r and armv8-a. After
this commit, the deduced "-march=" option
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Dominik Wójt (domin144)
Changes
The deduced "-march=" option always started with aarch64, which is not a valid
value. There was also no way to distinguish between armv8-r and armv8-a. After
this commit, the deduced "-march=" option will s
AaronBallman wrote:
> @rjmccall Are these tests adequate?
While we appreciate the enthusiasm, please only ping reviewers after about a
week of not hearing from them on a review
(https://llvm.org/docs/CodeReview.html#code-reviews-speed-and-reciprocity),
thanks!
https://github.com/llvm/llvm-pr
jhuber6 wrote:
> Are we assuming any particular relationship to __builtin_readcyclecounter in
> terms of scales etc?
>
> __builtin_readsteadycounter could be used to access x86 MPERF clock counters,
> but to access the corresponding APERF clock we'd then need a
> __builtin_readvariablecounter
usx95 wrote:
I updated the patch to work for all the initializations that were missing
cleanups due to branch in an expression.
https://github.com/llvm/llvm-project/pull/80698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
https://github.com/MDevereau ready_for_review
https://github.com/llvm/llvm-project/pull/78169
___
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: Matthew Devereau (MDevereau)
Changes
Create some multiclasses for FMLA/FMLS, MLAL and MLSL to reduce the size of
arm_sme.td
---
Patch is 36.16 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/78
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/80040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1,7 +1,7 @@
-// RUN: %clang_cc1 %s -verify -fsyntax-only
+// RUN: %clang_cc1 -Wfree-nonheap-object -fsyntax-only -verify %s
void c1(int *a);
-
+typedef unsigned long size_t;
AaronBallman wrote:
```suggestion
typedef __typeof__(sizeof(0)) size_t;
```
https
https://github.com/AaronBallman commented:
Precommit CI found a valid failure on Windows:
```
FAIL: Clang :: Sema/attr-cleanup.c (15237 of 19292)
TEST 'Clang :: Sema/attr-cleanup.c' FAILED
Exit Code: 1
Command Output (stdout):
--
# RUN: at line 1
c:\ws\
https://github.com/usx95 updated https://github.com/llvm/llvm-project/pull/80698
>From 28d90711ff8e4924135d4bd4e5f252d96ac41b93 Mon Sep 17 00:00:00 2001
From: Utkarsh Saxena
Date: Tue, 23 Jan 2024 20:18:25 +
Subject: [PATCH 01/13] [misc-coroutine-hostile-raii] Use getOperand instead of
getC
AaronBallman wrote:
> @AaronBallman I am a little confused about what is the correct behavior for
> one the the cases in the test file for c.
>
> ```c++
> enum PR4515 {PR4515a=1u,PR4515b=(PR4515a-2)/2};
> int CheckPR4515[PR4515b==0?1:-1];
> ```
>
> At present since everything is ints this work
https://github.com/usx95 edited https://github.com/llvm/llvm-project/pull/80698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/usx95 edited https://github.com/llvm/llvm-project/pull/80698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/81331
>From 30341079e795c2668588b791f2c97b44006e7a04 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Fri, 9 Feb 2024 16:13:42 -0600
Subject: [PATCH] [WIP][LLVM] Add `__builtin_readsteadycounter` intrinsic and
buiilt
https://github.com/usx95 edited https://github.com/llvm/llvm-project/pull/80698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hokein wrote:
This change introduced an asan crash when running the
`QualifierFixerTest.IsQualifierType` unittest:
```
$ tools/clang/unittests/Format/FormatTests
--gtest_filter="QualifierFixerTest.IsQualifierType"
Note: Google Test filter = QualifierFixerTest.IsQualifierType
[==] Runni
Stylie777 wrote:
I think this can be one commit, the formatting changes can be squashed into the
commit that makes the changes. Also make sure the commit message fits in with
LLVM guidelines. See: https://llvm.org/docs/DeveloperPolicy.html#id18
https://github.com/llvm/llvm-project/pull/81466
_
Author: Joseph Huber
Date: 2024-02-12T08:15:48-06:00
New Revision: f5fd0deb2371d0bae3bef2563f50e005a140fc6d
URL:
https://github.com/llvm/llvm-project/commit/f5fd0deb2371d0bae3bef2563f50e005a140fc6d
DIFF:
https://github.com/llvm/llvm-project/commit/f5fd0deb2371d0bae3bef2563f50e005a140fc6d.diff
https://github.com/AtariDreams updated
https://github.com/llvm/llvm-project/pull/81335
>From 6103fff6c5040e70fa28bd088f8eafbe479cbab1 Mon Sep 17 00:00:00 2001
From: Rose <83477269+ataridre...@users.noreply.github.com>
Date: Fri, 9 Feb 2024 17:51:15 -0500
Subject: [PATCH 1/2] [ObjC] Add pre-commi
AaronBallman wrote:
> > > @Sirraide, would you be comfortable if you, @erichkeane, and I
> > > co-authored an RFC to the community?
> >
> >
> > Sure, I’d be happy to help with that as best I can. An RFC makes sense imo
> > seeing as this really impacts more parts of LLVM than just Clang.
>
>
Author: Haojian Wu
Date: 2024-02-12T15:24:27+01:00
New Revision: 50ed98f1cf2516c59de0351fa683639c937782ea
URL:
https://github.com/llvm/llvm-project/commit/50ed98f1cf2516c59de0351fa683639c937782ea
DIFF:
https://github.com/llvm/llvm-project/commit/50ed98f1cf2516c59de0351fa683639c937782ea.diff
LO
hokein wrote:
I have reverted it in 50ed98f1cf2516c59de0351fa683639c937782ea, feel free to
reland it with a fix.
https://github.com/llvm/llvm-project/pull/80241
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
https://github.com/hnakamura5 closed
https://github.com/llvm/llvm-project/pull/80299
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Hirofumi Nakamura
Date: 2024-02-12T23:27:09+09:00
New Revision: 6a471611a4b98396abb0680229aa0694ffe88a43
URL:
https://github.com/llvm/llvm-project/commit/6a471611a4b98396abb0680229aa0694ffe88a43
DIFF:
https://github.com/llvm/llvm-project/commit/6a471611a4b98396abb0680229aa0694ffe88a43.d
Author: Balázs Kéri
Date: 2024-02-12T15:30:58+01:00
New Revision: 2b5e4eeb26ce57a8d7c2835cffc80388f43f044a
URL:
https://github.com/llvm/llvm-project/commit/2b5e4eeb26ce57a8d7c2835cffc80388f43f044a
DIFF:
https://github.com/llvm/llvm-project/commit/2b5e4eeb26ce57a8d7c2835cffc80388f43f044a.diff
L
https://github.com/balazske closed
https://github.com/llvm/llvm-project/pull/81138
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -33,6 +33,7 @@ namespace clang {
class Decl;
struct DeducedPack;
class Sema;
+enum class TemplateDeductionResult;
erichkeane wrote:
should we specify the underlying type in all of these? I wonder if we can just
specify it as `uint8_t`? Would mean not hav
@@ -11445,11 +11452,16 @@ static void DiagnoseBadDeduction(Sema &S, NamedDecl
*Found, Decl *Templated,
(ParamD = Param.dyn_cast()) ||
(ParamD = Param.dyn_cast()) ||
(ParamD = Param.dyn_cast());
- switch (DeductionFailure.Result) {
- case Sema::TDK_Success:
-llvm_un
Author: Timm Bäder
Date: 2024-02-12T15:36:07+01:00
New Revision: 15b0cc1212701908400e8059c6581ffe85d8070f
URL:
https://github.com/llvm/llvm-project/commit/15b0cc1212701908400e8059c6581ffe85d8070f
DIFF:
https://github.com/llvm/llvm-project/commit/15b0cc1212701908400e8059c6581ffe85d8070f.diff
LO
Author: Timm Bäder
Date: 2024-02-12T15:36:07+01:00
New Revision: ea8de6e4336cf82aa541c6ad951b62585c3ea55c
URL:
https://github.com/llvm/llvm-project/commit/ea8de6e4336cf82aa541c6ad951b62585c3ea55c
DIFF:
https://github.com/llvm/llvm-project/commit/ea8de6e4336cf82aa541c6ad951b62585c3ea55c.diff
LO
https://github.com/AaronBallman approved this pull request.
LGTM! Precommit CI didn't run for Windows, but I tested the changes locally and
all tests came back clean for me.
https://github.com/llvm/llvm-project/pull/71709
___
cfe-commits mailing list
https://github.com/AaronBallman commented:
Thank you for this improvement! Please add a release note to
`clang/docs/ReleaseNotes.rst` so users know about the new diagnostic group.
https://github.com/llvm/llvm-project/pull/81364
___
cfe-commits mailing
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/81364
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -6165,6 +6165,10 @@ def ext_initializer_string_for_char_array_too_long :
ExtWarn<
def warn_missing_field_initializers : Warning<
"missing field %0 initializer">,
InGroup, DefaultIgnore;
+// The same warning, but another group is needed to disable it separately.
+def war
@@ -2363,23 +2356,32 @@ void InitListChecker::CheckStructUnionTypes(
}
// Emit warnings for missing struct field initializers.
- if (!VerifyOnly && InitializedSomething && CheckForMissingFields &&
- !RD->isUnion()) {
-// It is possible we have one or more unnamed
hnakamura5 wrote:
Thank you very much!
Currently I think this will be the largest part. I try to make so. Thank you
again for this, and of course for the other parts.
https://github.com/llvm/llvm-project/pull/80299
___
cfe-commits mailing list
cfe-com
jcsxky wrote:
@cor3ntin @erichkeane Could you please take another look at this PR? I think
current version would be a reasonable fix since it transforms type only when
it's needed. Or you have any other opinions? Thank you for guidance!
https://github.com/llvm/llvm-project/pull/80802
_
Author: Timm Bäder
Date: 2024-02-12T15:36:07+01:00
New Revision: 635dfd5d69c6be52a8a2bb612f3483d9a1226cfa
URL:
https://github.com/llvm/llvm-project/commit/635dfd5d69c6be52a8a2bb612f3483d9a1226cfa
DIFF:
https://github.com/llvm/llvm-project/commit/635dfd5d69c6be52a8a2bb612f3483d9a1226cfa.diff
LO
=?utf-8?q?“Nhat?= ,
=?utf-8?q?“Nhat?= ,
=?utf-8?q?“Nhat?= ,
=?utf-8?q?“Nhat?= ,
=?utf-8?q?“Nhat?= ,Nhat Nguyen
Message-ID:
In-Reply-To:
erichkeane wrote:
Looks like all the tests disappeared? Only thing I see is the code change.
Also, no release note is currently present.
https://github.c
JonChesterfield wrote:
New intrinsic sounds right - a constant frequency counter is a different thing
to a variable frequency counter.
"Steady" implies unchanging, so I'd agree with `readfixedfreqtimer` or similar.
We can't have a ratio between the two counters since one changes frequency and
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/81298
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane commented:
This would need a release note + I think it needs better testing.
@AaronBallman might wish to take a look as well, this is his code ownership.
https://github.com/llvm/llvm-project/pull/81298
___
cfe-commits ma
1 - 100 of 422 matches
Mail list logo
