[Openmp-commits] [PATCH] D60972: [OpenMP 5.0] libomptarget interface for declare mapper functions
Jonas Hahnfeld via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Mon Jun 10 04:41:52 PDT 2019
Hahnfeld added a comment.
In D60972#1533013 <https://reviews.llvm.org/D60972#1533013>, @lildmh wrote:
> @Hahnfeld Do you really think it is necessary to pass these two functions as arguments, instead of exporting them. If you do, could you explain why?
I don't say it's necessary, but rather an option. Asked differently, why do we need to export them (and decide on externally visible names) if they're only used in a function where we can pass them as arguments?
At the moment, this patch doesn't really add any value (meaning changed behavior) to the runtime. It's needed for D59474 <https://reviews.llvm.org/D59474> which is already large enough on its own, but it doesn't get to any state where it can be used.
CHANGES SINCE LAST ACTION
More information about the Openmp-commits