https://github.com/martinboehme closed
https://github.com/llvm/llvm-project/pull/72987
___
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 49f0070237709a979905d8fe823f47a1b0ebd8ee
674ade1adf2f66d81c8bf940479798a78ab32cb8 --
DonatNagyE wrote:
In fact, as I think about this, I realized that it's probably a bad idea to
perform "initialization" for each `va_list` variable in a
`check::PreStmt` callback -- e.g. because in theory the programmer
could write code like
```
void f(int fst, ...) {
va_list *pva = (va_list
bgra8 wrote:
@yxsamliu we found another problem with the patch.
https://godbolt.org/z/5M9eexKKM
```
#include
template
class Abc {
public:
Abc();
~Abc();
private:
struct Impl;
std::unique_ptr impl_;
};
template
struct Abc::Impl {
int x;
};
template
Abc::Abc() : impl_{std::
DonatNagyE wrote:
..and it turns out that `ValistChecker` already has a function called
`ValistChecker::getVAListAsRegion` that handles different representations of
`va_list` under various systems. (I was not familiar with it because it was
added by @Xazax-hun after the initial commit https://
Author: Erich Keane
Date: 2023-11-21T07:46:12-08:00
New Revision: 147b38b1464cb39e763a9865c476d5c3d54bc619
URL:
https://github.com/llvm/llvm-project/commit/147b38b1464cb39e763a9865c476d5c3d54bc619
DIFF:
https://github.com/llvm/llvm-project/commit/147b38b1464cb39e763a9865c476d5c3d54bc619.diff
L
Author: Momchil Velikov
Date: 2023-11-21T16:44:14Z
New Revision: 28f62d72f4d56de0db0ed20c9b8c309ec5e8e193
URL:
https://github.com/llvm/llvm-project/commit/28f62d72f4d56de0db0ed20c9b8c309ec5e8e193
DIFF:
https://github.com/llvm/llvm-project/commit/28f62d72f4d56de0db0ed20c9b8c309ec5e8e193.diff
LO
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Erich Keane (erichkeane)
Changes
`atomic` is required to be followed by a special `atomic clause`, so this patch
manages the parsing of that. We are representing each of the variants of the
atomic construct as separate kinds, because the
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Egor Zhdan (egorzhdan)
Changes
This upstreams more of the Clang API Notes functionality that is currently
implemented in the Apple fork:
https://github.com/apple/llvm-project/tree/next/clang/lib/APINotes
This is the largest chunk of the
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 2fe24fdc94cb10795a196eadcf3eed8055ea4000
b3b1a6c3c6774bc4158d7e273abcfbac5b5db02d --
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Mariya Podchishchaeva (Fznamznon)
Changes
Due to d0d2ee0e4bbe915d649e983c12d37bcfcf58823c clang doesn't perform qualified
name lookup into the current instantiation when it has dependent bases, because
of that `getTypeName` call always re
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Youngsuk Kim (JOE1994)
Changes
Opaque ptr cleanup effort
---
Full diff: https://github.com/llvm/llvm-project/pull/73020.diff
2 Files Affected:
- (modified) clang/lib/CodeGen/CGBuiltin.cpp (+4-9)
- (modified) clang/lib/CodeGen/C
DanielKristofKiss wrote:
I proposed a change to the aarch64 spec to help the harmonisation.
https://github.com/ARM-software/acle/pull/277
I'd drop the aarch64 specifics. I have a few more lines to make it work on
aarch64 (e.g. AppendTargetVersionMangling and AppendTargetClonesMangling needs
d
pirama-arumuga-nainar wrote:
@vitalybuka can you help review this change?
https://github.com/llvm/llvm-project/pull/66414
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -15,38 +20,57 @@ int main(void) {
if (__builtin_cpu_supports("sse4.2"))
a("sse4.2");
- // CHECK: [[LOAD:%[^ ]+]] = load i32, ptr getelementptr inbounds ({ i32,
i32, i32, [1 x i32] }, ptr @__cpu_model, i32 0, i32 3, i32 0)
- // CHECK: [[AND:%[^ ]+]] = and i32 [[LOAD
@@ -15,38 +20,57 @@ int main(void) {
if (__builtin_cpu_supports("sse4.2"))
a("sse4.2");
- // CHECK: [[LOAD:%[^ ]+]] = load i32, ptr getelementptr inbounds ({ i32,
i32, i32, [1 x i32] }, ptr @__cpu_model, i32 0, i32 3, i32 0)
- // CHECK: [[AND:%[^ ]+]] = and i32 [[LOAD
@@ -15,38 +20,57 @@ int main(void) {
if (__builtin_cpu_supports("sse4.2"))
a("sse4.2");
- // CHECK: [[LOAD:%[^ ]+]] = load i32, ptr getelementptr inbounds ({ i32,
i32, i32, [1 x i32] }, ptr @__cpu_model, i32 0, i32 3, i32 0)
- // CHECK: [[AND:%[^ ]+]] = and i32 [[LOAD
@@ -15,38 +20,57 @@ int main(void) {
if (__builtin_cpu_supports("sse4.2"))
a("sse4.2");
- // CHECK: [[LOAD:%[^ ]+]] = load i32, ptr getelementptr inbounds ({ i32,
i32, i32, [1 x i32] }, ptr @__cpu_model, i32 0, i32 3, i32 0)
- // CHECK: [[AND:%[^ ]+]] = and i32 [[LOAD
@@ -15,38 +20,57 @@ int main(void) {
if (__builtin_cpu_supports("sse4.2"))
a("sse4.2");
- // CHECK: [[LOAD:%[^ ]+]] = load i32, ptr getelementptr inbounds ({ i32,
i32, i32, [1 x i32] }, ptr @__cpu_model, i32 0, i32 3, i32 0)
- // CHECK: [[AND:%[^ ]+]] = and i32 [[LOAD
Author: Erich Keane
Date: 2023-11-21T11:08:48-08:00
New Revision: 04e9b61cec8491ca35ba804d8e004a37caa4045e
URL:
https://github.com/llvm/llvm-project/commit/04e9b61cec8491ca35ba804d8e004a37caa4045e
DIFF:
https://github.com/llvm/llvm-project/commit/04e9b61cec8491ca35ba804d8e004a37caa4045e.diff
L
Author: bd1976bris
Date: 2023-11-21T11:46:34-08:00
New Revision: 36e53885e6ac52ff948fac909178f82b602477dd
URL:
https://github.com/llvm/llvm-project/commit/36e53885e6ac52ff948fac909178f82b602477dd
DIFF:
https://github.com/llvm/llvm-project/commit/36e53885e6ac52ff948fac909178f82b602477dd.diff
LO
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-driver
Author: Joseph Huber (jhuber6)
Changes
Summary:
We support standalone compilation for the NVPTX architecture using
'nvlink' as our linker. Because of the special handling required to
transform input files to cubins
Author: Logikable
Date: 2023-11-21T16:00:31-05:00
New Revision: 752c21be68613f92e2de16cd380098cf830bc261
URL:
https://github.com/llvm/llvm-project/commit/752c21be68613f92e2de16cd380098cf830bc261
DIFF:
https://github.com/llvm/llvm-project/commit/752c21be68613f92e2de16cd380098cf830bc261.diff
LOG
Author: Evgenii Stepanov
Date: 2023-11-21T13:11:12-08:00
New Revision: fb57f4e0e0b302ec1b3181e952a4bd4b3c57a286
URL:
https://github.com/llvm/llvm-project/commit/fb57f4e0e0b302ec1b3181e952a4bd4b3c57a286
DIFF:
https://github.com/llvm/llvm-project/commit/fb57f4e0e0b302ec1b3181e952a4bd4b3c57a286.di
Author: GkvJwa
Date: 2023-11-21T15:33:16-08:00
New Revision: 7c3c243c9bf80377fcad6c7699dc9aaedd650a18
URL:
https://github.com/llvm/llvm-project/commit/7c3c243c9bf80377fcad6c7699dc9aaedd650a18
DIFF:
https://github.com/llvm/llvm-project/commit/7c3c243c9bf80377fcad6c7699dc9aaedd650a18.diff
LOG: S
alexfh wrote:
@yxsamliu What's the plan here? This issue is blocking us. If there is no
obvious fix very soon, we need to revert this.
https://github.com/llvm/llvm-project/pull/72394
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://list
https://github.com/hstk30-hw edited
https://github.com/llvm/llvm-project/pull/72197
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -307,7 +307,12 @@ AArch64ABIInfo::classifyArgumentType(QualType Ty, bool
IsVariadic,
// 0.
if (IsEmpty && Size == 0)
return ABIArgInfo::getIgnore();
-return ABIArgInfo::getDirect(llvm::Type::getInt8Ty(getVMContext()));
+// An empty struct can have size
bd1976bris wrote:
Thanks for the reviews 👍
https://github.com/llvm/llvm-project/pull/72452
___
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-tidy
@llvm/pr-subscribers-clang
Author: Félix-Antoine Constantin (felix642)
Changes
This checks find usages of the inline keywork where it is already implicitly
defined by the compiler and suggests it's removal.
Fixes #72397
---
Full diff: https
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Ben Shi (benshi001)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/73072.diff
1 Files Affected:
- (modified) clang/lib/StaticAnalyzer/Checkers/StreamChecker.cpp (+3-2)
``diff
diff --git a/clang/lib/StaticAn
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Yuxuan Chen (yuxuanchen1997)
Changes
This change aims to fix an ICE in issue
https://github.com/llvm/llvm-project/issues/63803
The crash happens in `ExitCXXTryStmt` because `EmitAnyExpr()` adds additional
cleanup to the `EHScopeS
mzyKi wrote:
> ..and it turns out that `ValistChecker` already has a function called
> `ValistChecker::getVAListAsRegion` that handles different representations of
> `va_list` under various systems. (I was not familiar with it because it was
> added by @Xazax-hun after the initial commit https
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: dong jianqiang (dongjianqiang2)
Changes
Adds a warning, issued by the clang semantic analysis. The patch warns on
switch which don't have the default branch.
This is a counterpart of gcc's Wswitch-default.
---
Full diff: https://github.c
Author: Yuxuan Chen
Date: 2023-11-21T21:21:27-08:00
New Revision: 1fad78b123d20db675d339053e4265aceb07c4af
URL:
https://github.com/llvm/llvm-project/commit/1fad78b123d20db675d339053e4265aceb07c4af
DIFF:
https://github.com/llvm/llvm-project/commit/1fad78b123d20db675d339053e4265aceb07c4af.diff
L
https://github.com/antangelo created
https://github.com/llvm/llvm-project/pull/73087
Reland of f418319730341e9d41ce8ead6fbfe5603c343985 with proper handling of
template constructors
When a nested template is instantiated, the template pattern of the inner class
is not copied into the outer cl
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (antangelo)
Changes
Reland of f418319730341e9d41ce8ead6fbfe5603c343985 with proper handling of
template constructors
When a nested template is instantiated, the template pattern of the inner class
is not copied into the outer class
antangelo wrote:
Thank you for the reproducers! I have posted a candidate reland that addresses
both of these issues here: https://github.com/llvm/llvm-project/pull/73087
https://github.com/llvm/llvm-project/pull/69676
___
cfe-commits mailing list
cfe
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jianjian Guan (jacquesguan)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/73091.diff
1 Files Affected:
- (modified) clang/lib/AST/ItaniumMangle.cpp (+1-1)
``diff
diff --git a/clang/lib/AST/ItaniumMangle.cp
@@ -311,7 +318,10 @@ computeBlockInputState(const CFGBlock &Block,
AnalysisContext &AC) {
}
}
- JoinedStateBuilder Builder(AC);
+ // When performing the join, only retain state for those expressions that are
+ // consumed by this block. This avoids performing joins a
https://github.com/martinboehme edited
https://github.com/llvm/llvm-project/pull/72850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
martinboehme wrote:
> +1 for the simpler approach. I have a high-level question about the stability
> of the locations, its effect on convergence.
Answered in the inline comment.
> But I am happy with the `ExprToVal` part of the simple patch.
SG! Then let's continue the discussion on #72985?
Author: Sander de Smalen
Date: 2023-11-22T08:52:53Z
New Revision: 81b7f115fb272ef6fd6967f4121b64814b4bcf10
URL:
https://github.com/llvm/llvm-project/commit/81b7f115fb272ef6fd6967f4121b64814b4bcf10
DIFF:
https://github.com/llvm/llvm-project/commit/81b7f115fb272ef6fd6967f4121b64814b4bcf10.diff
L
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Qizhi Hu (jcsxky)
Changes
This patch aims to fix [error in ast-merge to new ast
file](https://github.com/llvm/llvm-project/issues/72783).
`ASTUnit` is put in `for` body and AST nodes would be deallocated by allocator.
Using these nodes la
https://github.com/simpal01 closed
https://github.com/llvm/llvm-project/pull/71545
___
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: Mariya Podchishchaeva (Fznamznon)
Changes
The implementation mostly reuses C++ code paths where possible, including
narrowing check in order to provide diagnostic messages in case initializer for
constexpr variable is not exactly represen
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 74cdb8e6f8c88b97204f540601a553b412d1cd56
1d70b7726e7d1f11622a6d5c8246b0737e024c8d --
Author: Jonas Paulsson
Date: 2023-11-22T11:24:45+01:00
New Revision: 2e09ea65c4f9e458ffb31fe63d84a4991704f9e6
URL:
https://github.com/llvm/llvm-project/commit/2e09ea65c4f9e458ffb31fe63d84a4991704f9e6
DIFF:
https://github.com/llvm/llvm-project/commit/2e09ea65c4f9e458ffb31fe63d84a4991704f9e6.diff
martinboehme wrote:
Update: I've run some tests on an internal codebase, and with this PR, Crubit's
nullability check sees a reduction in SAT solver timeouts of over 40% and a
reduction in "reached maximum iterations" errors of over 60% -- similar to the
previous more complicated approach in #
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
philnik777 wrote:
@PiotrZSL could you use the "Start review" button to avoid spamming people with
dozens of E-Mails for a single review?
https://github.com/llvm/llvm-project/pull/73069
__
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/73103
https://wiki.edg.com/pub/Wg21kona2023/StrawPolls/p2308r1.html
This implements P2308R1 as a DR and resolves CWG2459, CWG2450 and CWG2049.
>From 90a9c92ce532af7c6346a381499780f158cd26ca Mon Sep 17 00:00:00 2001
F
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: cor3ntin (cor3ntin)
Changes
https://wiki.edg.com/pub/Wg21kona2023/StrawPolls/p2308r1.html
This implements P2308R1 as a DR and resolves CWG2459, CWG2450 and CWG2049.
---
Full diff: https://github.com/llvm/llvm-project/pull/73103.diff
11
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/73105
… Enumerations
https://isocpp.org/files/papers/P2864R2.pdf
>From f548a8e11c75b3157a9337892688f5414234bff8 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Wed, 22 Nov 2023 12:25:20 +0100
Subject: [PATCH] [C
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: cor3ntin (cor3ntin)
Changes
… Enumerations
https://isocpp.org/files/papers/P2864R2.pdf
---
Full diff: https://github.com/llvm/llvm-project/pull/73105.diff
5 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+3)
- (modified) cl
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
philnik777 wrote:
> > @PiotrZSL could you use the "Start review" button to avoid spamming people
> > with dozens of E-Mails for a single review?
>
> @philnik777 If you prefer not to receive notifications, simply unsubscribe.
> Spe
https://github.com/huixie90 updated
https://github.com/llvm/llvm-project/pull/68597
>From 617d6d23b2f861cd6dceb82f54a2685059b6 Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Thu, 14 Sep 2023 09:09:29 +
Subject: [PATCH] [flang] Enable alias tags pass by default
Enable by default when o
Author: Utkarsh Saxena
Date: 2023-11-22T12:51:24+01:00
New Revision: fbfd2c9972695bc74a646bfa5e3e2d00477dabe2
URL:
https://github.com/llvm/llvm-project/commit/fbfd2c9972695bc74a646bfa5e3e2d00477dabe2
DIFF:
https://github.com/llvm/llvm-project/commit/fbfd2c9972695bc74a646bfa5e3e2d00477dabe2.diff
llvmbot wrote:
@llvm/pr-subscribers-mc
@llvm/pr-subscribers-backend-aarch64
Author: None (CarolineConcatto)
Changes
…d LLVM intrinisc (#70362)""
This reverts commit e1ee0e85104eed2c68b6821d9e5a2066e4154099.
The patch https://github.com/llvm/llvm-project/pull/70362 had a test in LLDB
fai
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (CarolineConcatto)
Changes
…d LLVM intrinisc (#70362)""
This reverts commit e1ee0e85104eed2c68b6821d9e5a2066e4154099.
The patch https://github.com/llvm/llvm-project/pull/70362 had a test in LLDB
failing. The Feature sve2p1 in AArch6
llvmbot wrote:
@llvm/pr-subscribers-flang-driver
Author: Tom Eccles (tblah)
Changes
Enable by default when optimizing for speed.
For simplicity, only forward the flag to the frontend driver when it
contradicts what is implied by the optimization level.
Since https://github.com/llvm/llvm
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/73105
>From 6c02a92f2d2dbd426bf9e5fd34cc0f32203a0283 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Wed, 22 Nov 2023 12:25:20 +0100
Subject: [PATCH] [Clang] Implement P28646R2 Remove Deprecated Arithmetic
Conver
@@ -1355,6 +1362,10 @@ unsigned UnwrappedLineFormatter::format(
bool FixIndentation = (FixBadIndentation || ContinueFormatting) &&
Indent != TheLine.First->OriginalColumn;
bool ShouldFormat = TheLine.Affected || FixIndentation;
+
+if (Style
https://github.com/tomekpaszek updated
https://github.com/llvm/llvm-project/pull/70338
>From b5ba0b3fde2c6662e19dfdf96a787621ec767460 Mon Sep 17 00:00:00 2001
From: Tomek Paszek
Date: Sat, 11 Nov 2023 19:38:00 +0100
Subject: [PATCH 01/11] Added an option to ignore macro definitions.
---
clang
https://github.com/CarolineConcatto closed
https://github.com/llvm/llvm-project/pull/73110
___
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-tidy
Author: Björn Svensson (bjosv)
Changes
This check implements the [rule
17.5.1](https://www.perforce.com/resources/qac/high-integrity-cpp-coding-standard/standard-library)
of the HICPP standard which states:
- Do not ignore the result of std
https://github.com/qcolombet approved this pull request.
Thanks for your patience iterating through this.
LGTM
https://github.com/llvm/llvm-project/pull/70778
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailm
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Egor Zhdan (egorzhdan)
Changes
This upstreams more of the Clang API Notes functionality that is currently
implemented in the Apple fork:
https://github.com/apple/llvm-project/tree/next/clang/lib/APINotes
---
Full diff: https://git
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Michael Klemm (mjklemm)
Changes
The flang driver was silently ignoring the `main()` function in
`Fortran_main.a` for entry into the Fortran program unit if an external
`main()` as supplied (e.g., via cross-language linkage with Fortran an
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Michael Klemm (mjklemm)
Changes
The flang driver was silently ignoring the `main()` function in
`Fortran_main.a` for entry into the Fortran program unit if an external
`main()` as supplied (e.g., via cross-language linkage with For
https://github.com/DianQK updated
https://github.com/llvm/llvm-project/pull/67885
>From 9c2ce941f6eebbc84d5ed0860d5b10f70dfecb85 Mon Sep 17 00:00:00 2001
From: DianQK
Date: Thu, 28 Sep 2023 18:06:52 -0600
Subject: [PATCH 1/5] [SimplifyCFG] Pre-commit test for `switchToLookupTable`
---
.../Sim
https://github.com/DianQK ready_for_review
https://github.com/llvm/llvm-project/pull/67885
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DianQK wrote:
I apologize for the delayed response to the PR. I have resolved all regressions.
When a signed max-min cannot construct a lookup table, fall back to the min
lookup table.
https://github.com/llvm/llvm-project/pull/67885
___
cfe-commits ma
@@ -6519,17 +6518,55 @@ static bool SwitchToLookupTable(SwitchInst *SI,
IRBuilder<> &Builder,
SmallDenseMap ResultTypes;
SmallVector PHIs;
- for (SwitchInst::CaseIt E = SI->case_end(); CI != E; ++CI) {
-ConstantInt *CaseVal = CI->getCaseValue();
-if (CaseVal->get
Author: Momchil Velikov
Date: 2023-11-22T14:38:13Z
New Revision: ed5404cd6b20ef6b54ef713399f03857990861a8
URL:
https://github.com/llvm/llvm-project/commit/ed5404cd6b20ef6b54ef713399f03857990861a8
DIFF:
https://github.com/llvm/llvm-project/commit/ed5404cd6b20ef6b54ef713399f03857990861a8.diff
LO
https://github.com/DianQK updated
https://github.com/llvm/llvm-project/pull/67885
>From 9c2ce941f6eebbc84d5ed0860d5b10f70dfecb85 Mon Sep 17 00:00:00 2001
From: DianQK
Date: Thu, 28 Sep 2023 18:06:52 -0600
Subject: [PATCH 1/6] [SimplifyCFG] Pre-commit test for `switchToLookupTable`
---
.../Sim
@@ -6519,17 +6518,60 @@ static bool SwitchToLookupTable(SwitchInst *SI,
IRBuilder<> &Builder,
SmallDenseMap ResultTypes;
SmallVector PHIs;
- for (SwitchInst::CaseIt E = SI->case_end(); CI != E; ++CI) {
-ConstantInt *CaseVal = CI->getCaseValue();
-if (CaseVal->get
DanielKristofKiss wrote:
@efriedma-quic Sounds good to me, I'll land those change top of this PR and
drop these todos.
https://github.com/llvm/llvm-project/pull/71706
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-b
https://github.com/DonatNagyE requested changes to this pull request.
I don't think that this is a correct change, as according to the C standard "On
failure, the ftell function returns −1L and stores an implementation-defined
positive value in errno." [1]
Note that this is different from the
https://github.com/DonatNagyE edited
https://github.com/llvm/llvm-project/pull/73072
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
EugeneZelenko wrote:
I think `modernize` is better fit for this check.
https://github.com/llvm/llvm-project/pull/72385
___
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
@llvm/pr-subscribers-objectyaml
@llvm/pr-subscribers-backend-amdgpu
Author: Jay Foad (jayfoad)
Changes
Define target names and ELF numbers for new GFX12 targets gfx1200 and
gfx1201. For now they behave identically to GFX11.
---
Patch is 40.39 Ki
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Jay Foad (jayfoad)
Changes
Define target names and ELF numbers for new GFX12 targets gfx1200 and
gfx1201. For now they behave identically to GFX11.
---
Patch is 40.39 KiB, truncated to 20.00 KiB below, full version:
https://gith
=?utf-8?q?Don=C3=A1t?= Nagy
Message-ID:
In-Reply-To:
DonatNagyE wrote:
@haoNoQ What do you think about this commit? May I merge it?
https://github.com/llvm/llvm-project/pull/72402
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists
Author: Tom Stellard
Date: 2023-11-22T07:10:39-08:00
New Revision: 64c0e86c9e07bd65ac7c584ca2e97769852616a5
URL:
https://github.com/llvm/llvm-project/commit/64c0e86c9e07bd65ac7c584ca2e97769852616a5
DIFF:
https://github.com/llvm/llvm-project/commit/64c0e86c9e07bd65ac7c584ca2e97769852616a5.diff
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jonas Paulsson (JonPsson1)
Changes
- Clang FE now has MaxAtomicPromoteWidth and MaxAtomicInlineWidth with a value
of 128. It now produces IR instead of calls to __atomic instrinsics for 16
bytes as well. FP loads are first loaded as i128
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 9ceea088592ebaafed8eaccfc236a0ec284655ce
1011b8e7da174146dfb4c9a4bf54468ea021 --
EugeneZelenko wrote:
@firewave, @carlosgalvezp: What do you think?
https://github.com/llvm/llvm-project/pull/72385
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/martinboehme updated
https://github.com/llvm/llvm-project/pull/72985
>From 4b7bfb0093f8b02ba13f0b3ccaaeef26d7617b6a Mon Sep 17 00:00:00 2001
From: Martin Braenne
Date: Tue, 21 Nov 2023 14:25:57 +
Subject: [PATCH] [clang][dataflow] Clear `ExprToLoc` and `ExprToVal` at the
LWenH wrote:
Thank you also for your careful suggestions too.
https://github.com/llvm/llvm-project/pull/70778
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: martinboehme
Date: 2023-11-22T16:34:24+01:00
New Revision: c4c59192e6e913dcb0c68ad1632efb277d329cc7
URL:
https://github.com/llvm/llvm-project/commit/c4c59192e6e913dcb0c68ad1632efb277d329cc7
DIFF:
https://github.com/llvm/llvm-project/commit/c4c59192e6e913dcb0c68ad1632efb277d329cc7.diff
https://github.com/martinboehme closed
https://github.com/llvm/llvm-project/pull/72985
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/martinboehme updated
https://github.com/llvm/llvm-project/pull/73111
>From 617d6d23b2f861cd6dceb82f54a2685059b6 Mon Sep 17 00:00:00 2001
From: Tom Eccles
Date: Thu, 14 Sep 2023 09:09:29 +
Subject: [PATCH 1/2] [flang] Enable alias tags pass by default
Enable by defaul
martinboehme wrote:
Abandoning in favor of #72985.
https://github.com/llvm/llvm-project/pull/72850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/martinboehme closed
https://github.com/llvm/llvm-project/pull/72850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
https://github.com/EugeneZelenko edited
https://github.com/llvm/llvm-project/pull/73069
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
=?utf-8?q?F=C3=A9lix-Antoine?= Constantin
Message-ID:
In-Reply-To:
https://github.com/EugeneZelenko requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/73069
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:
=?utf-8?q?F=C3=A9lix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -0,0 +1,35 @@
+.. title:: clang-tidy - readability-redundant-inline-specifier
+
+readability-redundant-inline-specifier
+==
+
+Checks for instances of the `inline` keyword i
=?utf-8?q?Félix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -0,0 +1,99 @@
+//===--- RedundantInlineSpecifierCheck.cpp -
+// clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/
=?utf-8?q?F=C3=A9lix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -0,0 +1,35 @@
+.. title:: clang-tidy - readability-redundant-inline-specifier
+
+readability-redundant-inline-specifier
+==
+
+Checks for instances of the `inline` keyword i
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
2101 - 2200 of 402847 matches
Mail list logo
