================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to