https://github.com/luporl approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/140092
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-com
https://github.com/luporl edited
https://github.com/llvm/llvm-project/pull/140092
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -230,8 +230,15 @@ bool DataSharingProcessor::needBarrier() {
return false;
}
-void DataSharingProcessor::insertBarrier() {
- if (needBarrier())
+void DataSharingProcessor::insertBarrier(
+mlir::omp::PrivateClauseOps *clauseOps) {
luporl wrote:
```su
luporl wrote:
This also broke the build of
https://lab.llvm.org/buildbot/#/builders/141/builds/4432.
https://github.com/llvm/llvm-project/pull/83237
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin
luporl wrote:
Closing this, as it seems a new PR will be needed later, to merge
https://github.com/llvm/llvm-project/pull/101009 instead.
https://github.com/llvm/llvm-project/pull/100352
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.l
https://github.com/luporl closed
https://github.com/llvm/llvm-project/pull/100352
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
luporl wrote:
#95799 caused errors in some internal test suites and was reverted for now.
https://github.com/llvm/llvm-project/pull/100352
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
https://github.com/luporl converted_to_draft
https://github.com/llvm/llvm-project/pull/100352
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -2012,34 +2012,87 @@ void OmpAttributeVisitor::Post(const parser::Name
&name) {
}
}
}
-std::vector defaultDSASymbols;
+
+// Implicitly determined DSAs
+// OMP 5.2 5.1.1 - Variables Referenced in a Construct
+Symbol *lastDeclSymbol = nullptr
https://github.com/luporl updated
https://github.com/llvm/llvm-project/pull/85989
>From 94301e00239b789cf90d5291b1d733f0f2baab6c Mon Sep 17 00:00:00 2001
From: Leandro Lupori
Date: Mon, 11 Mar 2024 16:47:47 -0300
Subject: [PATCH 1/2] [flang][OpenMP] Support tasks' implicit firstprivate DSA
Han
https://github.com/luporl approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/87086
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
luporl wrote:
@NimishMishra Thanks for the review!
It seems the buildbot errors on Windows are all of this type: `fatal error
C1002: compiler is out of heap space in pass 2`.
> Is this merge not in main?
It is not. It depends on https://github.com/llvm/llvm-project/pull/85978, that
depends o
https://github.com/luporl created
https://github.com/llvm/llvm-project/pull/85989
Handle implicit firstprivate DSAs on task generating constructs.
Fixes https://github.com/llvm/llvm-project/issues/64480
>From 94301e00239b789cf90d5291b1d733f0f2baab6c Mon Sep 17 00:00:00 2001
From: Leandro Lupo
luporl wrote:
This PR depends on https://github.com/llvm/llvm-project/pull/72510.
https://github.com/llvm/llvm-project/pull/85978
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/ll
https://github.com/luporl created
https://github.com/llvm/llvm-project/pull/85978
Split nested default clause tests into multiple subroutines, to
make it easier to find failures. While here, fix indentation of
the modified lines.
>From 39b3e13e42cd9416101ddb7b5cbb4137e5bd66f3 Mon Sep 17 00:00:
@@ -1092,6 +1040,79 @@ class FirConverter : public
Fortran::lower::AbstractConverter {
return true;
}
+ void copyVar(const Fortran::semantics::Symbol &sym,
+ const Fortran::lower::SymbolBox &lhs_sb,
+ const Fortran::lower::SymbolBox &rhs_sb)
https://github.com/luporl created
https://github.com/llvm/llvm-project/pull/80488
Use the new copyprivate list from omp.single to emit calls to
__kmpc_copyprivate, during the creation of the single operation
in OMPIRBuilder.
This is patch 4 of 4, to add support for COPYPRIVATE in Flang.
Origina
https://github.com/luporl created
https://github.com/llvm/llvm-project/pull/80485
Add initial handling of OpenMP copyprivate clause in Flang.
When lowering copyprivate, Flang generates the copy function
needed by each variable and builds the appropriate
omp.single's CopyPrivateVarList.
This is
Author: Leandro Lupori
Date: 2023-04-27T11:10:43Z
New Revision: bd6783b380768bd35f37e0034dccf6c5736dd031
URL:
https://github.com/llvm/llvm-project/commit/bd6783b380768bd35f37e0034dccf6c5736dd031
DIFF:
https://github.com/llvm/llvm-project/commit/bd6783b380768bd35f37e0034dccf6c5736dd031.diff
LOG
Author: Leandro Lupori
Date: 2023-04-27T11:10:43Z
New Revision: ccbab5979b7bd594314b50f4fc54ec57a676f391
URL:
https://github.com/llvm/llvm-project/commit/ccbab5979b7bd594314b50f4fc54ec57a676f391
DIFF:
https://github.com/llvm/llvm-project/commit/ccbab5979b7bd594314b50f4fc54ec57a676f391.diff
LOG
20 matches
Mail list logo