Author: Chris B
Date: 2024-09-26T19:34:17-05:00
New Revision: d5ee252c6bddcd27210e3a031cef8065679c8da5
URL:
https://github.com/llvm/llvm-project/commit/d5ee252c6bddcd27210e3a031cef8065679c8da5
DIFF:
https://github.com/llvm/llvm-project/commit/d5ee252c6bddcd27210e3a031cef8065679c8da5.diff
LOG:
tgross35 wrote:
> Currently, Rust's compiler-builtins has marked fp16 as available for
> loongarch64, but in fact, the functionality is broken. Even with this patch,
> it is not optimal. Subjectively, I hope these patches can be backported to
> LLVM 19 to avoid ABI incompatibility issues acros
ldionne wrote:
@tru At this point, the important CI has passed. Please consider merging this
so I can rebase the other libc++ PRs on top of it, get the CI to run and clear
the backlog for the 19 milestone PRs.
https://github.com/llvm/llvm-project/pull/110162
___
Author: Jorge Gorbe Moya
Date: 2024-09-26T14:55:52-07:00
New Revision: 4b2d897e9dcacf206ccd2b2326621a25d11727e5
URL:
https://github.com/llvm/llvm-project/commit/4b2d897e9dcacf206ccd2b2326621a25d11727e5
DIFF:
https://github.com/llvm/llvm-project/commit/4b2d897e9dcacf206ccd2b2326621a25d11727e5.di
atrosinenko wrote:
Thank you for the review!
> As you are adding a new function, it's good time to fix the naming case to
> `emitToStreamer`
I think renaming `EmitToStreamer` to `emitToStreamer` now may be a bit
error-prone, as this function overrides the non-virtual
`AsmPrinter::EmitToStrea
https://github.com/shiltian edited
https://github.com/llvm/llvm-project/pull/109295
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/99891
>From 36197b175681d07b4704e576fb008cec3cc1e05e Mon Sep 17 00:00:00 2001
From: Amir Ayupov
Date: Wed, 28 Aug 2024 21:10:25 +0200
Subject: [PATCH 1/3] Reworked block probe matching
Use new probe ifaces
Get all func
https://github.com/aaupov edited https://github.com/llvm/llvm-project/pull/99891
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/ldionne milestoned
https://github.com/llvm/llvm-project/pull/110162
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
llvmbot wrote:
@llvm/pr-subscribers-libcxx
Author: Louis Dionne (ldionne)
Changes
The CI has been a complete mess for the past week, and the only thing
preventing it from being back is the Clang tidy checks. Disable them (as a
total hack) to get CI back.
(cherry picked from commit 78c65
ldionne wrote:
@tru This patch should make the libc++ CI pass again. Unfortunately, it looks
like we're going to have to let the clang-tidy checks go for now.
Once this passes CI (🤞), we can merge this, and then I can rebase #110161,
#109324, #109291, #109054 and #106077 which should all pass.
https://github.com/ldionne edited
https://github.com/llvm/llvm-project/pull/110162
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/ldionne created
https://github.com/llvm/llvm-project/pull/110162
The CI has been a complete mess for the past week, and the only thing
preventing it from being back is the Clang tidy checks. Disable them (as a
total hack) to get CI back.
(cherry picked from commit 78c650654
efriedma-quic wrote:
> (which for some reason is not emitted as an atomic).
In terms of the generated code, I think we're fine; we don't rely on the load
producing a value that's consistent with atomic ordering, I think.
That said, it probably should be atomic, because strictly speaking a race
https://github.com/aaupov edited https://github.com/llvm/llvm-project/pull/99891
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/efriedma-quic commented:
I'm not sure we can assume all metadata which applies to an atomicrmw also
makes sense on the generated cmpxchg. I mean, most metadata probably does, but
say we start allowing `!align` metadata on atomicrmw...
https://github.com/llvm/llvm-project/pu
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/99891
>From 36197b175681d07b4704e576fb008cec3cc1e05e Mon Sep 17 00:00:00 2001
From: Amir Ayupov
Date: Wed, 28 Aug 2024 21:10:25 +0200
Subject: [PATCH 1/2] Reworked block probe matching
Use new probe ifaces
Get all func
@@ -25,20 +25,29 @@ bool llvm::lowerAtomicCmpXchgInst(AtomicCmpXchgInst *CXI) {
Value *Cmp = CXI->getCompareOperand();
Value *Val = CXI->getNewValOperand();
- LoadInst *Orig =
- Builder.CreateAlignedLoad(Val->getType(), Ptr, CXI->getAlign());
- Value *Equal = Builde
@@ -0,0 +1,62 @@
+## Tests stale block matching with pseudo probes.
+
+# REQUIRES: system-linux
+# RUN: split-file %s %t
+# RUN: llvm-mc -filetype=obj -triple x86_64-unknown-unknown %t/main.s -o %t.o
+# RUN: %clang %cflags %t.o -o %t.exe -Wl,-q -nostdlib
+# RUN: llvm-bolt %t.exe -
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/99891
>From 36197b175681d07b4704e576fb008cec3cc1e05e Mon Sep 17 00:00:00 2001
From: Amir Ayupov
Date: Wed, 28 Aug 2024 21:10:25 +0200
Subject: [PATCH 1/2] Reworked block probe matching
Use new probe ifaces
Get all func
@@ -722,6 +770,15 @@ Error YAMLProfileReader::readProfile(BinaryContext &BC) {
}
}
+ if (opts::StaleMatchingWithPseudoProbes) {
+const MCPseudoProbeDecoder *Decoder = BC.getPseudoProbeDecoder();
+assert(Decoder &&
+ "If pseudo probes are in use, pseudo
Author: Han-Chung Wang
Date: 2024-09-26T11:17:47-07:00
New Revision: 61e03b68511862ace5bc361565b2535e27fde833
URL:
https://github.com/llvm/llvm-project/commit/61e03b68511862ace5bc361565b2535e27fde833
DIFF:
https://github.com/llvm/llvm-project/commit/61e03b68511862ace5bc361565b2535e27fde833.diff
https://github.com/aaupov edited https://github.com/llvm/llvm-project/pull/99891
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/aaupov edited https://github.com/llvm/llvm-project/pull/99891
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/MaskRay approved this pull request.
As you are adding a new function, it's good time to fix the naming case to
`emitToStreamer`
https://github.com/llvm/llvm-project/pull/110107
___
llvm-branch-commits mailing list
llvm-branch-commit
https://github.com/kparzysz updated
https://github.com/llvm/llvm-project/pull/110015
>From 3c786ad2a50f146d357d882b0c1d966486f7295f Mon Sep 17 00:00:00 2001
From: Krzysztof Parzyszek
Date: Tue, 24 Sep 2024 17:41:16 -0500
Subject: [PATCH 1/5] [flang][OpenMP] Add version checks for clauses
If th
@@ -163,6 +163,43 @@ class AssociatedLoopChecker {
std::map constructNamesAndLevels_;
};
+bool OmpStructureChecker::CheckAllowedClause(llvmOmpClause clause) {
+ unsigned version{context_.langOptions().OpenMPVersion};
+ DirectiveContext &dirCtx = GetContext();
+ llvm::omp:
https://github.com/kovdan01 commented:
LGTM, but please wait for other reviews
https://github.com/llvm/llvm-project/pull/110108
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm
https://github.com/tblah approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/110013
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-comm
llvmbot wrote:
@llvm/pr-subscribers-backend-aarch64
Author: Anatoly Trosinenko (atrosinenko)
Changes
Most of PAuth-related code counts the instructions being inserted and asserts
that no more bytes are emitted than the size returned by the
getInstSizeInBytes(MI) method. This check seems
https://github.com/atrosinenko created
https://github.com/llvm/llvm-project/pull/110108
Most of PAuth-related code counts the instructions being inserted and asserts
that no more bytes are emitted than the size returned by the
getInstSizeInBytes(MI) method. This check seems useful not only for
llvmbot wrote:
@llvm/pr-subscribers-backend-aarch64
Author: Anatoly Trosinenko (atrosinenko)
Changes
In preparation for refactoring the instruction size checks being made by
PAuth-related code, switch all instruction emission in AArch64AsmPrinter to
using EmitToStreamer function.
Introd
https://github.com/atrosinenko created
https://github.com/llvm/llvm-project/pull/110107
In preparation for refactoring the instruction size checks being made by
PAuth-related code, switch all instruction emission in AArch64AsmPrinter to
using EmitToStreamer function.
Introduce a single-operan
https://github.com/tblah approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/110015
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-comm
https://github.com/skatrak edited
https://github.com/llvm/llvm-project/pull/110015
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/skatrak approved this pull request.
Thank you Krzysztof, LGTM.
https://github.com/llvm/llvm-project/pull/110015
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/l
@@ -163,6 +163,43 @@ class AssociatedLoopChecker {
std::map constructNamesAndLevels_;
};
+bool OmpStructureChecker::CheckAllowedClause(llvmOmpClause clause) {
+ unsigned version{context_.langOptions().OpenMPVersion};
+ DirectiveContext &dirCtx = GetContext();
+ llvm::omp:
https://github.com/ergawy approved this pull request.
Nice! LGTM!
https://github.com/llvm/llvm-project/pull/110015
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commi
38 matches
Mail list logo
