[Lldb-commits] [lldb] [lldb][DWARFASTParserClang] Resolve nested types when parsing structures (PR #66879)

2023-10-15 Thread Vlad Serebrennikov via lldb-commits 

https://github.com/Endilll closed 
https://github.com/llvm/llvm-project/pull/66879
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [lldb][DWARFASTParserClang] Resolve nested types when parsing structures (PR #66879)

2023-10-15 Thread Vlad Serebrennikov via lldb-commits 

Endilll wrote:

Superseded by #68705

https://github.com/llvm/llvm-project/pull/66879
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [OpenMP][mlir] Added `num_teams`, `thread_limit` translation to LLVM IR (PR #68821)

2023-10-15 Thread via lldb-commits 

https://github.com/shraiysh updated 
https://github.com/llvm/llvm-project/pull/68821

>From fff04e51e5d799fd9f39e0d58fc3d1b854325981 Mon Sep 17 00:00:00 2001
From: Shraiysh Vaishay 
Date: Wed, 11 Oct 2023 11:51:19 -0500
Subject: [PATCH] [OpenMP][mlir] Added `num_teams`, `thread_limit` translation
 to LLVM IR

This patch adds translation to LLVM IR for `num_teams` and
`thread_limit` in for `omp.teams` operation.
---
 .../OpenMP/OpenMPToLLVMIRTranslation.cpp  |  22 +++-
 mlir/test/Target/LLVMIR/openmp-teams.mlir | 111 ++
 2 files changed, 127 insertions(+), 6 deletions(-)

diff --git 
a/mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp 
b/mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
index 1ec3bb8e7562a9e..ae974c14fac41a6 100644
--- a/mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
+++ b/mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
@@ -667,11 +667,9 @@ convertOmpTeams(omp::TeamsOp op, llvm::IRBuilderBase 
&builder,
 LLVM::ModuleTranslation &moduleTranslation) {
   using InsertPointTy = llvm::OpenMPIRBuilder::InsertPointTy;
   LogicalResult bodyGenStatus = success();
-  if (op.getNumTeamsLower() || op.getNumTeamsUpper() || op.getIfExpr() ||
-  op.getThreadLimit() || !op.getAllocatorsVars().empty() ||
-  op.getReductions()) {
+  if (op.getIfExpr() || !op.getAllocatorsVars().empty() || op.getReductions())
 return op.emitError("unhandled clauses for translation to LLVM IR");
-  }
+
   auto bodyCB = [&](InsertPointTy allocaIP, InsertPointTy codegenIP) {
 LLVM::ModuleTranslation::SaveStack frame(
 moduleTranslation, allocaIP);
@@ -680,9 +678,21 @@ convertOmpTeams(omp::TeamsOp op, llvm::IRBuilderBase 
&builder,
 moduleTranslation, bodyGenStatus);
   };
 
+  llvm::Value *numTeamsLower = nullptr;
+  if (auto numTeamsLowerVar = op.getNumTeamsLower())
+numTeamsLower = moduleTranslation.lookupValue(numTeamsLowerVar);
+
+  llvm::Value *numTeamsUpper = nullptr;
+  if (auto numTeamsUpperVar = op.getNumTeamsUpper())
+numTeamsUpper = moduleTranslation.lookupValue(numTeamsUpperVar);
+
+  llvm::Value *threadLimit = nullptr;
+  if (auto threadLimitVar = op.getThreadLimit())
+threadLimit = moduleTranslation.lookupValue(threadLimitVar);
+
   llvm::OpenMPIRBuilder::LocationDescription ompLoc(builder);
-  builder.restoreIP(
-  moduleTranslation.getOpenMPBuilder()->createTeams(ompLoc, bodyCB));
+  builder.restoreIP(moduleTranslation.getOpenMPBuilder()->createTeams(
+  ompLoc, bodyCB, numTeamsLower, numTeamsUpper, threadLimit));
   return bodyGenStatus;
 }
 
diff --git a/mlir/test/Target/LLVMIR/openmp-teams.mlir 
b/mlir/test/Target/LLVMIR/openmp-teams.mlir
index 18fc2bb5a3c61b2..87ef90223ed704a 100644
--- a/mlir/test/Target/LLVMIR/openmp-teams.mlir
+++ b/mlir/test/Target/LLVMIR/openmp-teams.mlir
@@ -124,3 +124,114 @@ llvm.func @omp_teams_branching_shared(%condition: i1, 
%arg0: i32, %arg1: f32, %a
 // CHECK-NEXT: br label
 // CHECK: ret void
 
+// -
+
+llvm.func @beforeTeams()
+llvm.func @duringTeams()
+llvm.func @afterTeams()
+
+// CHECK-LABEL: @omp_teams_thread_limit
+// CHECK-SAME: (i32 [[THREAD_LIMIT:.+]])
+llvm.func @omp_teams_thread_limit(%threadLimit: i32) {
+// CHECK-NEXT: call void @beforeTeams()
+llvm.call @beforeTeams() : () -> ()
+// CHECK: [[THREAD_NUM:%.+]] = call i32 @__kmpc_global_thread_num
+// CHECK-NEXT: call void @__kmpc_push_num_teams_51({{.+}}, i32 
[[THREAD_NUM]], i32 0, i32 0, i32 [[THREAD_LIMIT]])
+// CHECK: call void (ptr, i32, ptr, ...) @__kmpc_fork_teams(ptr @1, i32 0, 
ptr [[OUTLINED_FN:.+]])
+omp.teams thread_limit(%threadLimit : i32) {
+llvm.call @duringTeams() : () -> ()
+omp.terminator
+}
+// CHECK: call void @afterTeams
+llvm.call @afterTeams() : () -> ()
+// CHECK: ret void
+llvm.return
+}
+
+// CHECK: define internal void [[OUTLINED_FN]](ptr {{.+}}, ptr {{.+}})
+// CHECK: call void @duringTeams()
+// CHECK: ret void
+
+// -
+
+llvm.func @beforeTeams()
+llvm.func @duringTeams()
+llvm.func @afterTeams()
+
+// CHECK-LABEL: @omp_teams_num_teams_upper
+// CHECK-SAME: (i32 [[NUM_TEAMS_UPPER:.+]])
+llvm.func @omp_teams_num_teams_upper(%numTeamsUpper: i32) {
+// CHECK-NEXT: call void @beforeTeams()
+llvm.call @beforeTeams() : () -> ()
+// CHECK: [[THREAD_NUM:%.+]] = call i32 @__kmpc_global_thread_num
+// CHECK-NEXT: call void @__kmpc_push_num_teams_51({{.+}}, i32 
[[THREAD_NUM]], i32 [[NUM_TEAMS_UPPER]], i32 [[NUM_TEAMS_UPPER]], i32 0)
+// CHECK: call void (ptr, i32, ptr, ...) @__kmpc_fork_teams(ptr @1, i32 0, 
ptr [[OUTLINED_FN:.+]])
+omp.teams num_teams(to %numTeamsUpper : i32) {
+llvm.call @duringTeams() : () -> ()
+omp.terminator
+}
+// CHECK: call void @afterTeams
+llvm.call @afterTeams() : () -> ()
+// CHECK: ret void
+llvm.return
+}
+
+// CHECK: define internal void [[OUTLINED_FN]](ptr

[Lldb-commits] [lldb] [OpenMP][mlir] Added `num_teams`, `thread_limit` translation to LLVM IR (PR #68821)

2023-10-15 Thread via lldb-commits 

https://github.com/shraiysh updated 
https://github.com/llvm/llvm-project/pull/68821

>From fff04e51e5d799fd9f39e0d58fc3d1b854325981 Mon Sep 17 00:00:00 2001
From: Shraiysh Vaishay 
Date: Wed, 11 Oct 2023 11:51:19 -0500
Subject: [PATCH 1/2] [OpenMP][mlir] Added `num_teams`, `thread_limit`
 translation to LLVM IR

This patch adds translation to LLVM IR for `num_teams` and
`thread_limit` in for `omp.teams` operation.
---
 .../OpenMP/OpenMPToLLVMIRTranslation.cpp  |  22 +++-
 mlir/test/Target/LLVMIR/openmp-teams.mlir | 111 ++
 2 files changed, 127 insertions(+), 6 deletions(-)

diff --git 
a/mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp 
b/mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
index 1ec3bb8e7562a9e..ae974c14fac41a6 100644
--- a/mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
+++ b/mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp
@@ -667,11 +667,9 @@ convertOmpTeams(omp::TeamsOp op, llvm::IRBuilderBase 
&builder,
 LLVM::ModuleTranslation &moduleTranslation) {
   using InsertPointTy = llvm::OpenMPIRBuilder::InsertPointTy;
   LogicalResult bodyGenStatus = success();
-  if (op.getNumTeamsLower() || op.getNumTeamsUpper() || op.getIfExpr() ||
-  op.getThreadLimit() || !op.getAllocatorsVars().empty() ||
-  op.getReductions()) {
+  if (op.getIfExpr() || !op.getAllocatorsVars().empty() || op.getReductions())
 return op.emitError("unhandled clauses for translation to LLVM IR");
-  }
+
   auto bodyCB = [&](InsertPointTy allocaIP, InsertPointTy codegenIP) {
 LLVM::ModuleTranslation::SaveStack frame(
 moduleTranslation, allocaIP);
@@ -680,9 +678,21 @@ convertOmpTeams(omp::TeamsOp op, llvm::IRBuilderBase 
&builder,
 moduleTranslation, bodyGenStatus);
   };
 
+  llvm::Value *numTeamsLower = nullptr;
+  if (auto numTeamsLowerVar = op.getNumTeamsLower())
+numTeamsLower = moduleTranslation.lookupValue(numTeamsLowerVar);
+
+  llvm::Value *numTeamsUpper = nullptr;
+  if (auto numTeamsUpperVar = op.getNumTeamsUpper())
+numTeamsUpper = moduleTranslation.lookupValue(numTeamsUpperVar);
+
+  llvm::Value *threadLimit = nullptr;
+  if (auto threadLimitVar = op.getThreadLimit())
+threadLimit = moduleTranslation.lookupValue(threadLimitVar);
+
   llvm::OpenMPIRBuilder::LocationDescription ompLoc(builder);
-  builder.restoreIP(
-  moduleTranslation.getOpenMPBuilder()->createTeams(ompLoc, bodyCB));
+  builder.restoreIP(moduleTranslation.getOpenMPBuilder()->createTeams(
+  ompLoc, bodyCB, numTeamsLower, numTeamsUpper, threadLimit));
   return bodyGenStatus;
 }
 
diff --git a/mlir/test/Target/LLVMIR/openmp-teams.mlir 
b/mlir/test/Target/LLVMIR/openmp-teams.mlir
index 18fc2bb5a3c61b2..87ef90223ed704a 100644
--- a/mlir/test/Target/LLVMIR/openmp-teams.mlir
+++ b/mlir/test/Target/LLVMIR/openmp-teams.mlir
@@ -124,3 +124,114 @@ llvm.func @omp_teams_branching_shared(%condition: i1, 
%arg0: i32, %arg1: f32, %a
 // CHECK-NEXT: br label
 // CHECK: ret void
 
+// -
+
+llvm.func @beforeTeams()
+llvm.func @duringTeams()
+llvm.func @afterTeams()
+
+// CHECK-LABEL: @omp_teams_thread_limit
+// CHECK-SAME: (i32 [[THREAD_LIMIT:.+]])
+llvm.func @omp_teams_thread_limit(%threadLimit: i32) {
+// CHECK-NEXT: call void @beforeTeams()
+llvm.call @beforeTeams() : () -> ()
+// CHECK: [[THREAD_NUM:%.+]] = call i32 @__kmpc_global_thread_num
+// CHECK-NEXT: call void @__kmpc_push_num_teams_51({{.+}}, i32 
[[THREAD_NUM]], i32 0, i32 0, i32 [[THREAD_LIMIT]])
+// CHECK: call void (ptr, i32, ptr, ...) @__kmpc_fork_teams(ptr @1, i32 0, 
ptr [[OUTLINED_FN:.+]])
+omp.teams thread_limit(%threadLimit : i32) {
+llvm.call @duringTeams() : () -> ()
+omp.terminator
+}
+// CHECK: call void @afterTeams
+llvm.call @afterTeams() : () -> ()
+// CHECK: ret void
+llvm.return
+}
+
+// CHECK: define internal void [[OUTLINED_FN]](ptr {{.+}}, ptr {{.+}})
+// CHECK: call void @duringTeams()
+// CHECK: ret void
+
+// -
+
+llvm.func @beforeTeams()
+llvm.func @duringTeams()
+llvm.func @afterTeams()
+
+// CHECK-LABEL: @omp_teams_num_teams_upper
+// CHECK-SAME: (i32 [[NUM_TEAMS_UPPER:.+]])
+llvm.func @omp_teams_num_teams_upper(%numTeamsUpper: i32) {
+// CHECK-NEXT: call void @beforeTeams()
+llvm.call @beforeTeams() : () -> ()
+// CHECK: [[THREAD_NUM:%.+]] = call i32 @__kmpc_global_thread_num
+// CHECK-NEXT: call void @__kmpc_push_num_teams_51({{.+}}, i32 
[[THREAD_NUM]], i32 [[NUM_TEAMS_UPPER]], i32 [[NUM_TEAMS_UPPER]], i32 0)
+// CHECK: call void (ptr, i32, ptr, ...) @__kmpc_fork_teams(ptr @1, i32 0, 
ptr [[OUTLINED_FN:.+]])
+omp.teams num_teams(to %numTeamsUpper : i32) {
+llvm.call @duringTeams() : () -> ()
+omp.terminator
+}
+// CHECK: call void @afterTeams
+llvm.call @afterTeams() : () -> ()
+// CHECK: ret void
+llvm.return
+}
+
+// CHECK: define internal void [[OUTLINED_FN]]

[Lldb-commits] [lldb] [OpenMP][mlir] Added `num_teams`, `thread_limit` translation to LLVM IR (PR #68821)

2023-10-15 Thread via lldb-commits 

https://github.com/shraiysh closed 
https://github.com/llvm/llvm-project/pull/68821
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [lldb] [NVPTX] Improve lowering of v4i8 (PR #67866)

2023-10-15 Thread Andrew Adams via lldb-commits 

abadams wrote:

I believe this may be causing failures for u/srem. See #69124 

https://github.com/llvm/llvm-project/pull/67866
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits