[PATCH] D87946: [OpenMP] Add Location Fields to Libomptarget Runtime for Debugging

Joseph Huber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 23 06:28:23 PDT 2020


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



More information about the llvm-commits mailing list