mjklemm wrote:
@skatrak Is this ready for final review?
https://github.com/llvm/llvm-project/pull/92524
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/97564
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/97565
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/97718
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/107233
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/107234
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/107235
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -287,3 +287,117 @@ been implemented, but it is closely linked to the
`omp.canonical_loop` work.
Nevertheless, loop transformation that the `collapse` clause for
loop-associated
worksharing constructs defines can be represented by introducing multiple
bounds, step and induc
https://github.com/mjklemm approved this pull request.
https://github.com/llvm/llvm-project/pull/107236
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
mjklemm wrote:
Is this PR in an acceptable state to be merged? I went through the other three
and it seems that the 4 PRs all converging and seem either ready for merge or
very close.
https://github.com/llvm/llvm-project/pull/101446
___
llvm-branch-
mjklemm wrote:
> * The above requirement basically breaks allowing users to pick their own
> openmp library when compiling their application. I'm not sure how widely used
> this is.
They won't be able to that regardless. The OpenMP library will have to match
what Flang is using; otherwise, t
https://github.com/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/110266
___
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/mjklemm approved this pull request.
LGTM, albeit not being a true expert here. But the general idea looks fine to
me.
https://github.com/llvm/llvm-project/pull/116219
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.l
https://github.com/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/116049
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/117875
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/112188
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/146033
___
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/mjklemm approved this pull request.
LGTM, maybe fix the typo.
https://github.com/llvm/llvm-project/pull/142735
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/ll
@@ -308,10 +310,47 @@ class DoConcurrentConversion
fir::DoConcurrentLoopOp loop, mlir::IRMapping &mapper,
const mlir::omp::LoopNestOperands &clauseOps,
bool isComposite) const {
+mlir::omp::WsloopOperands wsloopClauseOps;
+
+//
https://github.com/mjklemm edited
https://github.com/llvm/llvm-project/pull/142735
___
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/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/141772
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -107,7 +107,7 @@ enum TargetAllocTy : int32_t {
inline KernelArgsTy CTorDTorKernelArgs = {1, 0, nullptr, nullptr,
nullptr, nullptr, nullptr, nullptr,
-0, {0,0,0}, {1, 0, 0}, {1, 0, 0}, 0};
+0, {0,0,0,0},
mjklemm wrote:
I have started to put the Fortran interfaces together in
https://github.com/mjklemm/llvm-project/tree/ftn-dyn-groupprivate-api. I will
add the Fortran API once the Fortran support has been done in Flang.
https://github.com/llvm/llvm-project/pull/152831
_
https://github.com/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/152830
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -4497,6 +4576,22 @@ void OmpStructureChecker::CheckWorkshareBlockStmts(
}
}
+void OmpStructureChecker::CheckWorkdistributeBlockStmts(
+const parser::Block &block, parser::CharBlock source) {
+ OmpWorkdistributeBlockChecker ompWorkdistributeBlockChecker{context_,
sou
https://github.com/mjklemm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/154378
___
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/mjklemm requested changes to this pull request.
Please slightly adjust the commented error message and put the `workdistribute`
under a version check for OpenMP 6.0.
https://github.com/llvm/llvm-project/pull/154377
___
llvm-branch-c
https://github.com/mjklemm edited
https://github.com/llvm/llvm-project/pull/154377
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
28 matches
Mail list logo