jhuber6 added a comment. In D87946#2289959 <https://reviews.llvm.org/D87946#2289959>, @grokos wrote:
> Yes, this used to be a point of contention within the community. We discussed > the issue sometime ago and the majority of developers was in favor of this > approach (as opposed to e.g. having an extra pointer to a structure which > will contain additional information and which will be extended every time we > add a new feature). > > The libomptarget-part of this patch looks good, I'm leaving the other > reviewers look at the clang-part. In the future we'll want the ability to pass the mapped variable's string name so we can refer to target pointers by name and line number in the runtime. Since this will require adding another argument containing a list of the names we'll probably want to try to add both at the same time to avoid the need for more suffixes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87946/new/ https://reviews.llvm.org/D87946 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
