[llvm-branch-commits] [flang] [Flang][OpenMP][MLIR] Initial derived type member map support (PR #82853)

via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Wed Apr 24 09:34:11 PDT 2024


================
@@ -185,7 +184,12 @@ template <typename T>
 bool ClauseProcessor::processMotionClauses(
----------------
agozillon wrote:

I agree and have been thinking about it, as there isn't a whole lot of dissimilarities, most changes to processMapClauses need replicated to processMotionClauses and it would make testing a lot simpler! But it'd also be up to @ergawy I am unsure if he'd like to keep them distinct and has some insights into motion clauses that might make merging the two functions unideal! 

But I can make a follow up PR after the stack has landed to merge them if we are all happy doing so.

https://github.com/llvm/llvm-project/pull/82853


More information about the llvm-branch-commits mailing list