https://github.com/owenca edited https://github.com/llvm/llvm-project/pull/73753
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NMiehlbradt updated
https://github.com/llvm/llvm-project/pull/73611
>From 9fb1d97a4fd3e9a314780f5961d9c394cd6ae20c Mon Sep 17 00:00:00 2001
From: Nicholas Miehlbradt
Date: Thu, 26 Oct 2023 00:45:22 +
Subject: [PATCH 1/2] [KMSAN] Enable on PowerPC64
Enable
@@ -0,0 +1,149 @@
+; RUN: opt < %s -S -msan-kernel=1 -passes=msan 2>&1 | FileCheck %s
+
+target datalayout = "e-m:e-i64:64-n32:64"
+target triple = "powerpc64le--linux"
+
+define void @Store1(ptr %p, i8 %x) sanitize_memory {
+entry:
+ store i8 %x, ptr %p
+ ret void
+}
+
+;
@@ -0,0 +1,149 @@
+; RUN: opt < %s -S -msan-kernel=1 -passes=msan 2>&1 | FileCheck %s
MaskRay wrote:
`llvm/test/Instrumentation/MemorySanitizer/PowerPC/vararg-ppc64.ll` does not
follow the convention. I think the new test should follow other tests which
follow
cor3ntin wrote:
@tbaederr ping :)
https://github.com/llvm/llvm-project/pull/70053
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
NMiehlbradt wrote:
@eugenis @ramosian-glider
https://github.com/llvm/llvm-project/pull/73611
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
@knightXun code looks reasonable, ping me when you have tests and a release
note :)
https://github.com/llvm/llvm-project/pull/70280
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
cor3ntin wrote:
That's a lot of repeated code. Should we have a "NoteTemplateLocation"
function(s) of some sort that would do the check and the diag as appropriate?
https://github.com/llvm/llvm-project/pull/71264
___
cfe-commits mailing list
https://github.com/owenca edited https://github.com/llvm/llvm-project/pull/73753
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca edited https://github.com/llvm/llvm-project/pull/73753
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/saiislam approved this pull request.
Thanks, LGTM!
https://github.com/llvm/llvm-project/pull/73709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DanShaders wrote:
@MaskRay, I figured the alternative solution with the completely decoupled
driver/frontend flags might actually be better than my initial approach. The
only issue I see with it is that it requires
`s/-mms-bitfields/-fms-layout-compatibility=microsoft/g` in quite a large
https://github.com/fodinabor created
https://github.com/llvm/llvm-project/pull/73756
As per https://github.com/AdaptiveCpp/AdaptiveCpp/issues/1256 - we are missing
the `fma` long double variant for Cpp20 compact with MS-STL.
See also #49853.
-> I'm wondering if we should include this for all
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: None (fodinabor)
Changes
As per https://github.com/AdaptiveCpp/AdaptiveCpp/issues/1256 - we are missing
the `fma` long double variant for Cpp20 compact with MS-STL.
See also #49853.
- I'm wondering if we should include this for all
https://github.com/fodinabor 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
@@ -0,0 +1,384 @@
+// NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
+// REQUIRES: aarch64-registered-target
+
+// RUN: %clang_cc1 -triple aarch64-none-linux-gnu -target-feature +sve
-target-feature +sme2 -S -disable-O0-optnone -Werror -Wall
david-arm wrote:
Should the file be renamed to acle_sme2_vector_selx4? This would make it
consistent with the existing acle_sme2_vector_add.c file, which also has
SVE-like instructions that only operate on SVE vectors.
david-arm wrote:
Should the file be renamed to acle_sme2_vector_selx2? This would make it
consistent with the existing acle_sme2_vector_add.c file, which also has
SVE-like instructions that only operate on SVE vectors.
https://github.com/Bigcheese created
https://github.com/llvm/llvm-project/pull/73719
The working directory is included in the PCM, but is not currently part of the
context hash. This causes problems because different builds of a PCM with
exactly the same command line can end up with different
Author: Alex Voicu
Date: 2023-11-29T00:15:18Z
New Revision: 57a0416e0e8ccd522d4242dbe5d0d7893864a10a
URL:
https://github.com/llvm/llvm-project/commit/57a0416e0e8ccd522d4242dbe5d0d7893864a10a
DIFF:
https://github.com/llvm/llvm-project/commit/57a0416e0e8ccd522d4242dbe5d0d7893864a10a.diff
LOG:
https://github.com/AlexVlx closed
https://github.com/llvm/llvm-project/pull/69266
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Bigcheese created
https://github.com/llvm/llvm-project/pull/73734
`-ivfsoverlay` files are unused when building most modules. Enable
removing them by,
* adding a way to visit the filesystem tree with extensible RTTI to
access each `RedirectingFileSystem`.
* Adding tracking
https://github.com/DanShaders updated
https://github.com/llvm/llvm-project/pull/71148
>From 1033441efd9e790bdb027824ffa1986cd09c06f2 Mon Sep 17 00:00:00 2001
From: Dan Klishch
Date: Fri, 3 Nov 2023 21:18:06 -0400
Subject: [PATCH 1/6] [clang] Stub out gcc_struct attribute
This commit
Author: Yuxuan Chen
Date: 2023-11-28T19:04:29-08:00
New Revision: 4a294b5806417aa88c91aa05735b2d557ea5dfe5
URL:
https://github.com/llvm/llvm-project/commit/4a294b5806417aa88c91aa05735b2d557ea5dfe5
DIFF:
https://github.com/llvm/llvm-project/commit/4a294b5806417aa88c91aa05735b2d557ea5dfe5.diff
https://github.com/yuxuanchen1997 closed
https://github.com/llvm/llvm-project/pull/73160
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Chuanqi Xu
Date: 2023-11-29T11:45:31+08:00
New Revision: 649e8111a95ae0d8814576e9ca74823572ee404b
URL:
https://github.com/llvm/llvm-project/commit/649e8111a95ae0d8814576e9ca74823572ee404b
DIFF:
https://github.com/llvm/llvm-project/commit/649e8111a95ae0d8814576e9ca74823572ee404b.diff
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Michael Spencer (Bigcheese)
Changes
The working directory is included in the PCM, but is not currently part of the
context hash. This causes problems because different builds of a PCM with
exactly the same command line can end up with
owenca wrote:
This patch makes `isProto()` a convenience function like `isCpp()`, which
doesn't prevent us from handling C++ and Objective-C separately when needed.
Although `LK_TextProto` is not always formatted like `LK_Proto`, it's frequent
enough (about 16 times in the codebase) that we
https://github.com/philnik777 reopened
https://github.com/llvm/llvm-project/pull/65148
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/65148
>From 02e9afd761228f401df4d9f8dfaaca44ffae0c6e Mon Sep 17 00:00:00 2001
From: zijunzhao
Date: Thu, 31 Aug 2023 20:08:32 +
Subject: [PATCH 01/17] [libc++] Implement ranges::contains
Differential Revision:
https://github.com/philnik777 closed
https://github.com/llvm/llvm-project/pull/70258
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
philnik777 wrote:
I've reopened the old one for you. Let's keep the discussion in one place.
https://github.com/llvm/llvm-project/pull/70258
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Bill Wendling (bwendling)
Changes
This is an alternative to https://github.com/llvm/llvm-project/pull/73465. It
generates the GEP directly. It's not tested well, but it might be a nicer
solution rather than adding AST nodes.
PTAL
---
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 4eb421192479dbecae2621b868e55aaf6d945b02
3e500c2a7c6b7895ebe292a1ed50e04409ba149c --
https://github.com/yuxuanchen1997 updated
https://github.com/llvm/llvm-project/pull/73564
>From 5f5ebec41c90366bf3c7ec1ee53154ba7afcb849 Mon Sep 17 00:00:00 2001
From: Yuxuan Chen
Date: Wed, 22 Nov 2023 13:38:54 -0800
Subject: [PATCH] Refactor how we generate RValue vs LValue coawait
https://github.com/ZijunZhaoCCK updated
https://github.com/llvm/llvm-project/pull/65148
>From 02e9afd761228f401df4d9f8dfaaca44ffae0c6e Mon Sep 17 00:00:00 2001
From: zijunzhao
Date: Thu, 31 Aug 2023 20:08:32 +
Subject: [PATCH 01/17] [libc++] Implement ranges::contains
Differential
https://github.com/bwendling created
https://github.com/llvm/llvm-project/pull/73730
This is an alternative to https://github.com/llvm/llvm-project/pull/73465. It
generates the GEP directly. It's not tested well, but it might be a nicer
solution rather than adding AST nodes.
PTAL
>From
llvmbot wrote:
@llvm/pr-subscribers-llvm-support
Author: Michael Spencer (Bigcheese)
Changes
`-ivfsoverlay` files are unused when building most modules. Enable
removing them by,
* adding a way to visit the filesystem tree with extensible RTTI to
access each `RedirectingFileSystem`.
*
@@ -0,0 +1,74 @@
+//===- DomConditionCache.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:
@@ -75,7 +75,8 @@ define void @idom_sign_bit_check_edge_dominates_select(i64
%a, i64 %b) {
; CHECK: land.lhs.true:
; CHECK-NEXT:br label [[LOR_END:%.*]]
; CHECK: lor.rhs:
-; CHECK-NEXT:[[CMP3_NOT:%.*]] = icmp eq i64 [[A]], [[B:%.*]]
+; CHECK-NEXT:
https://github.com/DanShaders created
https://github.com/llvm/llvm-project/pull/73737
This causes CI failure for PRs updating `AttrDocs.td`. CC @llvm-beanz
>From d30a251b2026065de3260f473b34c080102504e9 Mon Sep 17 00:00:00 2001
From: Dan Klishch
Date: Tue, 28 Nov 2023 22:04:47 -0500
Subject:
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Dan Klishch (DanShaders)
Changes
This causes CI failure for PRs updating `AttrDocs.td`. CC @llvm-beanz
---
Full diff: https://github.com/llvm/llvm-project/pull/73737.diff
1 Files Affected:
- (modified)
https://github.com/DanShaders edited
https://github.com/llvm/llvm-project/pull/73737
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
philnik777 wrote:
Would it make sense to document this more publicly? While it's designed to be
an internal tool, it's really useful for people who want to make sure their
library produces high quality diagnostics (e.g. `nodiscard`, `static_assert`s
etc.). I'm sure you are aware that libc++
https://github.com/bwendling updated
https://github.com/llvm/llvm-project/pull/73730
>From 3e500c2a7c6b7895ebe292a1ed50e04409ba149c Mon Sep 17 00:00:00 2001
From: Bill Wendling
Date: Tue, 28 Nov 2023 17:17:54 -0800
Subject: [PATCH 1/2] [Clang] Generate the GEP instead of adding AST nodes
---
https://github.com/jcsxky closed https://github.com/llvm/llvm-project/pull/73290
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Qizhi Hu
Date: 2023-11-29T09:33:37+08:00
New Revision: 3287ae8f6520ef81570377c1fb4c7147782a13ef
URL:
https://github.com/llvm/llvm-project/commit/3287ae8f6520ef81570377c1fb4c7147782a13ef
DIFF:
https://github.com/llvm/llvm-project/commit/3287ae8f6520ef81570377c1fb4c7147782a13ef.diff
https://github.com/ChuanqiXu9 approved this pull request.
LGTM with nit.
https://github.com/llvm/llvm-project/pull/73160
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
lifengxiang1025 wrote:
> Yes, you're right. As an alternative can we use the symbol table and find
> Bind = LOCAL to add the prefix before hashing?
If we choose this method. I think we can't deal with the situation which one
symbol is not local linkage type in thin compile, but will be
@@ -129,14 +129,48 @@ static SmallString<32> buildSuspendPrefixStr(CGCoroData
, AwaitKind Kind) {
return Prefix;
}
-static bool memberCallExpressionCanThrow(const Expr *E) {
- if (const auto *CE = dyn_cast(E))
-if (const auto *Proto =
-
https://github.com/ChuanqiXu9 edited
https://github.com/llvm/llvm-project/pull/73160
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
psteinfeld wrote:
> @mjklemm, after this change was integrated, the test Driver/ctofortran no
> longer succeeds. This test gets run when you call `make check-flang`. I'm not
> sure why the pre- and post-build checks did not run check-flang.
I've submitted #73738 to fix this.
https://github.com/owenca edited https://github.com/llvm/llvm-project/pull/73582
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Endilll approved this pull request.
https://github.com/llvm/llvm-project/pull/73493
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -824,20 +817,76 @@ void StreamChecker::evalFgetcFputc(const FnDescription
*Desc,
// If a (non-EOF) error occurs, the resulting value of the file position
// indicator for the stream is indeterminate.
- StreamErrorState NewES;
- if (IsRead)
-NewES =
-
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
steakhal wrote:
I'm in favor of this change. I'll pull the patch downstream and report back how
it performed.
Coming back to the `[size]` example, actually I believe
balazske wrote:
I prefer to have the new test code with `StreamTesterChecker_make_feof_stream`
in a new test function (with appropriate name like
`write_after_eof_is_allowed`), to improve code maintainability (`fputc` and
`fputs` can be in the same function).
In github PR's usually force
https://github.com/mjklemm updated
https://github.com/llvm/llvm-project/pull/73124
>From 2a2693364cb8e9b657b9ff54aa78df0466b55fe4 Mon Sep 17 00:00:00 2001
From: Michael Klemm
Date: Wed, 22 Nov 2023 14:22:20 +0100
Subject: [PATCH 01/13] Let the linker fail on multiple definitions of main()
---
mjklemm wrote:
Folks, I have made another attempt to improve this patch. @kparzysz with your
feedback in mind, I have now added a check if `--whole-archive` is active for
some reason. If so, flang will not add it to the link line again.
https://github.com/llvm/llvm-project/pull/73124
Endilll wrote:
> The human-readability of a big list of integers is not better than embedded
> base64 -- and actually, seems more of a pain to decode.
I agree that the entirety of the data is not too comprehensible, but I can
imagine users being interested in the first and last N bytes when
Author: Nikita Popov
Date: 2023-11-28T09:58:30+01:00
New Revision: b31cd07de5b7dfb435b062c80d85083e14f28321
URL:
https://github.com/llvm/llvm-project/commit/b31cd07de5b7dfb435b062c80d85083e14f28321
DIFF:
https://github.com/llvm/llvm-project/commit/b31cd07de5b7dfb435b062c80d85083e14f28321.diff
https://github.com/sdesmalen-arm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/72272
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rampitec updated
https://github.com/llvm/llvm-project/pull/72709
>From 423a0d1d4640680c5db3382ca0652fe85051ad8d Mon Sep 17 00:00:00 2001
From: Stanislav Mekhanoshin
Date: Fri, 17 Nov 2023 10:52:13 -0800
Subject: [PATCH] [AMDGPU] Fix folding of v2i16/v2f16 splat imms
We can
https://github.com/rampitec edited
https://github.com/llvm/llvm-project/pull/72709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
banach-space wrote:
Yes, the original comment in the file was not accurate - thanks for the
updating it!
However, as the name of the file suggests, "dynamic-linker.f90" tests for
behaviour relevant to the **dynamic** linker/linking. And,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
https://github.com/steakhal approved this pull request.
Overall, I'm in favor of this change.
On the other hand, I'd urge for not to regress on the diagnostics.
To me, `alloca` is like a VLA; which is prone to misuses,
301 - 366 of 366 matches
Mail list logo
