[PATCH] D86068: Python utils script extract_vplan.py to extract VPlan digraphs regexp fix

[Mauri Mustonen via Phabricator via llvm-commits]

Kazhuu created this revision.
Kazhuu added reviewers: rengolin, lattner.
Herald added subscribers: llvm-commits, psnobl, rogfer01, bollu.
Herald added a project: LLVM.
Kazhuu requested review of this revision.
Herald added a subscriber: vkmr.

Regarding this bug in Bugzilla: https://bugs.llvm.org/show_bug.cgi?id=46451

I went ahead and fixed the regexp pattern and now Python script is able to extract vplan graphs from the log files. Additionally some test for this would be nice to have but I'm not sure are Python scripts tested in LLVM and if so where they live.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D86068

Files:
  llvm/utils/extract_vplan.py


Index: llvm/utils/extract_vplan.py
===================================================================
--- llvm/utils/extract_vplan.py
+++ llvm/utils/extract_vplan.py
@@ -24,7 +24,7 @@
 matches = re.findall(pattern, sys.stdin.read())
 
 for vplan in matches:
-    m = re.search("graph \[.+(VF=.+,UF.+), ", vplan)
+    m = re.search("graph \[.+(VF=.+,UF.+)", vplan)
     if not m:
         raise ValueError("Can't get the right VPlan name")
     name = re.sub('[^a-zA-Z0-9]', '', m.group(1))


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86068.285986.patch
Type: text/x-patch
Size: 491 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200817/6fd8402d/attachment.bin>