[PATCH] D75591: [OpenMP] Add firstprivate as a default data-sharing attribute to clang
Johannes Doerfert via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 5 12:07:11 PST 2020
jdoerfert added inline comments.
================
Comment at: clang/test/OpenMP/parallel_master_codegen.cpp:143
+// CK31: [[A_VAL:%.+]] = alloca i32
+// CK31: call void (%struct.ident_t*, i32, void (i32*, i32*, ...)*, ...) @__kmpc_fork_call(%struct.ident_t* {{.+}}, i32 1, void (i32*, i32*, ...)* bitcast (void (i32*, i32*, i32*)* [[OMP_OUTLINED:@.+]] to void
+
----------------
This does not look like `firstprivate` is actually doing anything here. Could you please confirm that the IR is the same for `default(shared)` and `default(firstprivate)`? If so, we need to figure out why the variable is not marked firstprivate.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75591/new/
https://reviews.llvm.org/D75591
More information about the cfe-commits
mailing list