[llvm-branch-commits] [flang] [llvm] [flang][OpenMP] Sema checks, lowering with new format of MAP modifiers (PR #149137)

Tom Eccles via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Mon Jul 21 06:25:18 PDT 2025


================
@@ -1003,12 +1003,13 @@ Map make(const parser::OmpClause::Map &inp,
   CLAUSET_ENUM_CONVERT( //
       convert1, parser::OmpMapType::Value, Map::MapType,
----------------
tblah wrote:

Not new in this patch, but I don't really like the `convert1`  `convert2`  `convert3` naming. It makes the later code harder to follow than it needs to be.

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


More information about the llvm-branch-commits mailing list