Author: Ulrich Weigand
Date: 2023-12-04T16:52:00+01:00
New Revision: c61eb440059d6e9c18e6f8404e06bf125aa942c9
URL:
https://github.com/llvm/llvm-project/commit/c61eb440059d6e9c18e6f8404e06bf125aa942c9
DIFF:
https://github.com/llvm/llvm-project/commit/c61eb440059d6e9c18e6f8404e06bf125aa942c9.diff
MDevereau wrote:
> This looks good to me, but I think it needs rebasing after
> https://github.com/llvm/llvm-project/pull/72849 landed. It also looks like
> @sdesmalen-arm left a comment about renaming ImmToTile - perhaps that could
> be done in this patch?
I've rebased it and changed the fun
Author: Ulrich Weigand
Date: 2023-12-04T16:53:18+01:00
New Revision: dca432cb7b1c282f5dc861095813c4f40f109619
URL:
https://github.com/llvm/llvm-project/commit/dca432cb7b1c282f5dc861095813c4f40f109619
DIFF:
https://github.com/llvm/llvm-project/commit/dca432cb7b1c282f5dc861095813c4f40f109619.diff
Author: Ulrich Weigand
Date: 2023-12-04T17:00:05+01:00
New Revision: 9d27139293890a18b903c62f2c43954cc2bc302d
URL:
https://github.com/llvm/llvm-project/commit/9d27139293890a18b903c62f2c43954cc2bc302d
DIFF:
https://github.com/llvm/llvm-project/commit/9d27139293890a18b903c62f2c43954cc2bc302d.diff
Author: Da-Viper
Date: 2023-12-04T17:12:21+01:00
New Revision: 47a40e80c553b07a08f0f17fd68c12ae9b9a2a67
URL:
https://github.com/llvm/llvm-project/commit/47a40e80c553b07a08f0f17fd68c12ae9b9a2a67
DIFF:
https://github.com/llvm/llvm-project/commit/47a40e80c553b07a08f0f17fd68c12ae9b9a2a67.diff
LOG:
https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/69377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -751,7 +751,7 @@ This phase currently supports all the intrinsic procedures
listed above but the
| Object characteristic inquiry functions | ALLOCATED, ASSOCIATED,
EXTENDS_TYPE_OF, IS_CONTIGUOUS, PRESENT, RANK, SAME_TYPE, STORAGE_SIZE |
| Type inquiry intrinsic functions |
@@ -39,6 +39,17 @@
// overload will have a dummy parameter whose type indicates whether or not it
// should be preferred. Any other parameters required for SFINAE should have
// default values provided.
+
+// outside anonymous namespace, function reused
klausle
@@ -37,5 +80,17 @@ void FORTRAN_PROCEDURE_NAME(getarg)(
(void)RTNAME(GetCommandArgument)(
n, &value, nullptr, nullptr, __FILE__, __LINE__);
}
+
+void FORTRAN_PROCEDURE_NAME(getlog)(std::int8_t *arg, std::int64_t length) {
+ std::array str;
klausler wro
https://github.com/Fznamznon created
https://github.com/llvm/llvm-project/pull/74335
Current context set to where alias was met, not where it is declared caused
incorrect access check in case alias referenced private members of the parent
class.
Fixes https://github.com/llvm/llvm-project/issu
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Mariya Podchishchaeva (Fznamznon)
Changes
Current context set to where alias was met, not where it is declared caused
incorrect access check in case alias referenced private members of the parent
class.
Fixes https://github.com/llvm/llvm
Author: Momchil Velikov
Date: 2023-12-04T16:16:55Z
New Revision: e3a97dffee93f03e12b5911869f9a443c39821bc
URL:
https://github.com/llvm/llvm-project/commit/e3a97dffee93f03e12b5911869f9a443c39821bc
DIFF:
https://github.com/llvm/llvm-project/commit/e3a97dffee93f03e12b5911869f9a443c39821bc.diff
LO
https://github.com/momchil-velikov closed
https://github.com/llvm/llvm-project/pull/70565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/uweigand approved this pull request.
This version LGTM now.
https://github.com/llvm/llvm-project/pull/73134
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/74335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3990,9 +3990,14 @@ QualType Sema::CheckTemplateIdType(TemplateName Name,
if (Inst.isInvalid())
return QualType();
-CanonType = SubstType(Pattern->getUnderlyingType(),
- TemplateArgLists, AliasTemplate->getLocation(),
-
https://github.com/erichkeane approved this pull request.
I think this is on the right track, 1 nit, else is right I think.
https://github.com/llvm/llvm-project/pull/74335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
DonatNagyE wrote:
I tested this patch on four open source projects (openssl, ffmpeg, postgres,
contour) and the results were not affected (and there were no crashes). This is
not surprising, because released open source code doesn't contain too many bugs
and the message change only affects a s
Author: DonatNagyE
Date: 2023-12-04T17:19:50+01:00
New Revision: 67f387c67e2a1bfa9432cff372462e204e0952bc
URL:
https://github.com/llvm/llvm-project/commit/67f387c67e2a1bfa9432cff372462e204e0952bc
DIFF:
https://github.com/llvm/llvm-project/commit/67f387c67e2a1bfa9432cff372462e204e0952bc.diff
LO
https://github.com/DonatNagyE closed
https://github.com/llvm/llvm-project/pull/74141
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/67467
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL requested changes to this pull request.
Few fixes needed, but looks you are on good road.
https://github.com/llvm/llvm-project/pull/67467
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin
@@ -118,6 +119,7 @@ Clang-Tidy Checks
:doc:`bugprone-posix-return `, "Yes"
:doc:`bugprone-redundant-branch-condition
`, "Yes"
:doc:`bugprone-reserved-identifier `, "Yes"
+ :doc:`bugprone-shared-pointer-contents-move
`, "Yes"
PiotrZSL wrote:
reupli
@@ -0,0 +1,39 @@
+//===--- MoveSharedPointerContentsCheck.h - clang-tidy --*- C++
-*-===//
+//
+// 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
@@ -0,0 +1,51 @@
+//===--- MoveSharedPointerContentsCheck.cpp - clang-tidy
--===//
+//
+// 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
@@ -107,6 +107,7 @@ Clang-Tidy Checks
:doc:`bugprone-misplaced-pointer-arithmetic-in-alloc
`, "Yes"
:doc:`bugprone-misplaced-widening-cast `,
:doc:`bugprone-move-forwarding-reference
`, "Yes"
+ :doc:`bugprone-move-shared-pointer-contents
`, "Yes"
@@ -0,0 +1,51 @@
+//===--- MoveSharedPointerContentsCheck.cpp - clang-tidy
--===//
+//
+// 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
@@ -0,0 +1,51 @@
+//===--- MoveSharedPointerContentsCheck.cpp - clang-tidy
--===//
+//
+// 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
@@ -0,0 +1,51 @@
+//===--- MoveSharedPointerContentsCheck.cpp - clang-tidy
--===//
+//
+// 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
@@ -25,6 +25,7 @@ add_clang_library(clangTidyBugproneModule
ImplicitWideningOfMultiplicationResultCheck.cpp
InaccurateEraseCheck.cpp
IncorrectEnableIfCheck.cpp
+ MoveSharedPointerContentsCheck.cpp
PiotrZSL wrote:
Duplicated file, remove this line
https
@@ -0,0 +1,75 @@
+// RUN: %check_clang_tidy %s bugprone-move-shared-pointer-contents %t --
-config="{CheckOptions:
{bugprone-move-shared-pointer-contents.SharedPointerClasses:
'std::shared_ptr;my::OtherSharedPtr;'}}"
+
+// Some dummy definitions we'll need.
+
+namespace std {
+
Author: Thomas Schenker
Date: 2023-12-04T17:30:12+01:00
New Revision: 8ac84a9555bc25ed9a53ea6a5a09432e18ee9042
URL:
https://github.com/llvm/llvm-project/commit/8ac84a9555bc25ed9a53ea6a5a09432e18ee9042
DIFF:
https://github.com/llvm/llvm-project/commit/8ac84a9555bc25ed9a53ea6a5a09432e18ee9042.dif
https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/74215
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bazuzi created
https://github.com/llvm/llvm-project/pull/74336
…ng member pointers.
This initially landed with a broken test due to a mid-air collision with a new
requirement for Environment initialization before field modeling. Have added
that initialization in the test.
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Samira Bazuzi (bazuzi)
Changes
…ng member pointers.
This initially landed with a broken test due to a mid-air collision with a new
requirement for Environment initialization before field modeling. Have added
that initialization in the te
llvmbot wrote:
@llvm/pr-subscribers-clang-analysis
Author: Samira Bazuzi (bazuzi)
Changes
…ng member pointers.
This initially landed with a broken test due to a mid-air collision with a new
requirement for Environment initialization before field modeling. Have added
that initialization
bazuzi wrote:
@martinboehme or @ymand Could you take a look and merge?
The only modification from https://github.com/llvm/llvm-project/pull/73978 is
the `Env.initialize()` in the test.
https://github.com/llvm/llvm-project/pull/74336
___
cfe-commits m
https://github.com/Fznamznon updated
https://github.com/llvm/llvm-project/pull/74335
>From 6d7fe6fe8c62373fdaab597c3c31c7132e475e32 Mon Sep 17 00:00:00 2001
From: "Podchishchaeva, Mariya"
Date: Mon, 4 Dec 2023 07:47:04 -0800
Subject: [PATCH 1/2] [clang] Substitute alias templates from correct c
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/74335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vzakhari commented:
Thank you, Tom!
https://github.com/llvm/llvm-project/pull/74250
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nikic wrote:
@dtcxzyw Could you please run another test with this branch?
https://github.com/nikic/llvm-project/tree/perf/dom-cond-3
I believe the issue with DILATE is that we have some icmps that get
canonicalized from signed to unsigned predicate, and IndVars is not able to
perform some rea
Xazax-hun wrote:
Does this work with Z3 as the solver? Since Z3 is not officially supported, I
think it is not a blocker, but I'd love to see some FIXMEs/tickets opened in
that case.
https://github.com/llvm/llvm-project/pull/74141
___
cfe-commits ma
https://github.com/tblah created https://github.com/llvm/llvm-project/pull/74343
Now that tbaa tags pass is enabled by default, I would like to remove these
flags. `-fno-alias-analysis` was originally intended to be useful for
debugging, but as it also disables tbaa tag generation in codegen, i
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Tom Eccles (tblah)
Changes
Now that tbaa tags pass is enabled by default, I would like to remove these
flags. `-fno-alias-analysis` was originally intended to be useful for
debugging, but as it also disables tbaa tag generation in codegen
https://github.com/steakhal created
https://github.com/llvm/llvm-project/pull/74345
Review the commits one by one. I plan to merge them manually by pushing both of
these at once.
This PR intends to fix #74269.
>From 1359a7ef528358cc7e10a751aa885c6bd8ac8d1c Mon Sep 17 00:00:00 2001
From: Balazs
nikic wrote:
I went ahead and pushed a clean up version of that change to this PR, so you
can just re-test this PR.
I've found that dropping the icmp case also removes most of the second-order
compile-time regressions, and also drops the (very minor) regression in the BPF
test, so those are a
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Balazs Benics (steakhal)
Changes
Review the commits one by one. I plan to merge them manually by pushing both of
these at once.
This PR intends to fix #74269.
---
Patch is 40.76 KiB, truncated
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 f368e6424fbfb7fdea4b9d9a2e44f2f7e188c133
3575ad5701a852dd8b79d144ea17a729cfb4d94c --
topperc wrote:
> FWIW, Alive2 is complaining about this commit. These patches are not safe
> w.r.t. undef.
Why not?
https://github.com/llvm/llvm-project/pull/72912
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin
https://github.com/jyknight created
https://github.com/llvm/llvm-project/pull/74349
Update all callers to pass through the Address.
For the older builtins such as `__sync_*` and MSVC `_Interlocked*`, natural
alignment of the atomic access is _assumed_. This change preserves that
behavior. It
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: James Y Knight (jyknight)
Changes
Update all callers to pass through the Address.
For the older builtins such as `__sync_*` and MSVC `_Interlocked*`, natural
alignment of the atomic access is _assumed_. This change preserves that
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 f368e6424fbfb7fdea4b9d9a2e44f2f7e188c133
7fd7ebbf60beacb63ddfff16a7c4405e80cb62b4 --
jansvoboda11 wrote:
Ping @PiotrZSL.
https://github.com/llvm/llvm-project/pull/67839
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nunoplopes wrote:
Here's a simple-ish example:
```llvm
; Transforms/InstCombine/add.ll
define i5 @zext_sext_not(i4 %x) {
%zx = zext i4 %x to i5
%notx = xor i4 %x, 15
%snotx = sext i4 %notx to i5
%r = add i5 %zx, %snotx
ret i5 %r
}
=>
define i5 @zext_sext_not(i4 %x) {
%zx = zext i4 %x
https://github.com/madanial0 updated
https://github.com/llvm/llvm-project/pull/73254
>From 81d1e05dd084dd5bb88dab88d2f23008b8dc6cfb Mon Sep 17 00:00:00 2001
From: Mark Danial
Date: Tue, 21 Nov 2023 12:18:40 -0500
Subject: [PATCH 1/2] Pass the correct path to getIntriniscDir and
getOpenMPHeader
https://github.com/banach-space approved this pull request.
Thanks, LGTM!
https://github.com/llvm/llvm-project/pull/74343
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
asl wrote:
Looks like the review stalled. Where we are here? @ahmedbougacha @ChuanqiXu9 ?
https://github.com/llvm/llvm-project/pull/65996
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
https://github.com/pasaulais updated
https://github.com/llvm/llvm-project/pull/69229
>From 24b0e5204f9eac6efaf502b5aa254f5ff7e11e69 Mon Sep 17 00:00:00 2001
From: Pierre-Andre Saulais
Date: Thu, 23 Nov 2023 17:33:18 +
Subject: [PATCH] [AMDGPU] Add an option to disable unsafe uses of atomic
https://github.com/pasaulais updated
https://github.com/llvm/llvm-project/pull/69229
>From cd33e8471ab5e768c1b579f1bec1308c0b322e05 Mon Sep 17 00:00:00 2001
From: Pierre-Andre Saulais
Date: Thu, 23 Nov 2023 17:33:18 +
Subject: [PATCH] [AMDGPU] Add an option to disable unsafe uses of atomic
https://github.com/efriedma-quic approved this pull request.
LGTM
In terms of va_list etc., we first need to make sure calls are correct (compare
against gcc etc.), then we need to make sure va_list is consistent with that.
https://github.com/llvm/llvm-project/pull/72197
__
https://github.com/rjmccall approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/74349
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
=?utf-8?q?Andrés?= Villegas ,
=?utf-8?q?Andrés?= Villegas
Message-ID:
In-Reply-To:
https://github.com/avillega closed
https://github.com/llvm/llvm-project/pull/73194
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-b
https://github.com/jyknight updated
https://github.com/llvm/llvm-project/pull/74349
>From 7fd7ebbf60beacb63ddfff16a7c4405e80cb62b4 Mon Sep 17 00:00:00 2001
From: James Y Knight
Date: Mon, 4 Dec 2023 12:11:58 -0500
Subject: [PATCH 1/2] Use Address for CGBuilder's CreateAtomicRMW and
CreateAtomi
Author: James Y Knight
Date: 2023-12-04T13:37:04-05:00
New Revision: 4d4c30a37c75a4c41bcc70be1431651704624b6a
URL:
https://github.com/llvm/llvm-project/commit/4d4c30a37c75a4c41bcc70be1431651704624b6a
DIFF:
https://github.com/llvm/llvm-project/commit/4d4c30a37c75a4c41bcc70be1431651704624b6a.diff
https://github.com/jyknight closed
https://github.com/llvm/llvm-project/pull/74349
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ZijunZhaoCCK updated
https://github.com/llvm/llvm-project/pull/71771
>From 06c4cf02dfb4b20c8349c5f3c7209276f6d56edf Mon Sep 17 00:00:00 2001
From: zijunzhao
Date: Thu, 9 Nov 2023 02:21:46 +
Subject: [PATCH 1/4] Fix clang to recognize new C23 modifiers %w and %wf when
pri
@@ -70,6 +70,9 @@ __DEVICE__ double floor(double);
__DEVICE__ float floor(float);
__DEVICE__ double fma(double, double, double);
__DEVICE__ float fma(float, float, float);
+#ifdef _MSC_VER
+__DEVICE__ long double fma(long double, long double, long double);
Arte
https://github.com/Artem-B edited
https://github.com/llvm/llvm-project/pull/73756
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaronpuchert wrote:
> Aaron is the real decision maker here
Specifically @AaronBallman, not me.
https://github.com/llvm/llvm-project/pull/73077
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
https://github.com/mjklemm updated
https://github.com/llvm/llvm-project/pull/74139
>From 2e41335a7de3d2efa88eacee659172a3b9525e45 Mon Sep 17 00:00:00 2001
From: Michael Klemm
Date: Fri, 1 Dec 2023 21:41:44 +0100
Subject: [PATCH 1/6] Add -fno-fortran-main driver option
---
clang/include/clang/
301 - 370 of 370 matches
Mail list logo
