[Openmp-commits] [PATCH] D91085: [OpenMP] Begin Adding OpenMP Tool to Gather OpenMP Information

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

I think this is a good start, we can improve on it as we go. Please check if we need license information in the python files, e.g., check other python files. Also, one nit below.



================
Comment at: openmp/tools/analyzer/analyzer.py:28
+    def getLine(fn):
+        expr = re.compile("__omp_offloading_[a-zA-Z0-9]*_[a-zA-Z0-9]*__Z.*__l([0-9]*)$")
+        match = expr.search(fn)
----------------
Do we require _Z (here and above), isn't that only present for C++ ?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91085/new/

https://reviews.llvm.org/D91085