[PATCH] D59474: [OpenMP 5.0] Codegen support for user-defined mappers
Alexey Bataev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 26 13:46:27 PDT 2019
ABataev accepted this revision.
ABataev added a comment.
This revision is now accepted and ready to land.
Looks good in general, but commit the runtime part at first.
================
Comment at: test/OpenMP/declare_mapper_codegen.cpp:44-48
+// CK0-DAG: store i8* %0, i8** [[HANDLEADDR:%[^,]+]]
+// CK0-DAG: store i8* %1, i8** [[BPTRADDR:%[^,]+]]
+// CK0-DAG: store i8* %2, i8** [[VPTRADDR:%[^,]+]]
+// CK0-DAG: store i64 %3, i{{64|32}}* [[SIZEADDR:%[^,]+]]
+// CK0-DAG: store i64 %4, i64* [[TYPEADDR:%[^,]+]]
----------------
I would not rely on the predetermined indices here, better to use some kind of patterns here just like in other places.
================
Comment at: test/OpenMP/declare_mapper_codegen.cpp:289-293
+// CK1-DAG: store i8* %0, i8** [[HANDLEADDR:%[^,]+]]
+// CK1-DAG: store i8* %1, i8** [[BPTRADDR:%[^,]+]]
+// CK1-DAG: store i8* %2, i8** [[VPTRADDR:%[^,]+]]
+// CK1-DAG: store i64 %3, i{{64|32}}* [[SIZEADDR:%[^,]+]]
+// CK1-DAG: store i64 %4, i64* [[TYPEADDR:%[^,]+]]
----------------
Same here
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59474/new/
https://reviews.llvm.org/D59474
More information about the cfe-commits
mailing list