[llvm-bugs] [Bug 46451] New: Python script extract_vplan.py to extract VPlan digraphs regexp broken

[via llvm-bugs]

https://bugs.llvm.org/show_bug.cgi?id=46451

            Bug ID: 46451
           Summary: Python script extract_vplan.py to extract VPlan
                    digraphs regexp broken
           Product: new-bugs
           Version: 10.0
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P
         Component: new bugs
          Assignee: unassignedbugs at nondot.org
          Reporter: mauri_mustonen at hotmail.com
                CC: htmldeveloper at gmail.com, llvm-bugs at lists.llvm.org

Created attachment 23649
  --> https://bugs.llvm.org/attachment.cgi?id=23649&action=edit
vplan log

Hi all! My first issue here.

Python script in LLVM project called
extract_vplan.py:(https://github.com/llvm/llvm-project/blob/master/llvm/utils/extract_vplan.py#L27
is not able to extract VPlan graph because of regexp matching error. It seems
logging output has changed and regexp in the python script is not updated.
Following error is produced:
  File "../llvm/llvm/utils/extract_vplan.py", line 30, in <module>
    raise ValueError("Can't get the right VPlan name")
  ValueError: Can't get the right VPlan name

I added part of the log file as an attachment which can be used to test this.
To test it run this at llvm project root:
cat digraph.log | python utils/extract_vplan.py --png

When I change the regexp line:
- m = re.search("graph \[.+(VF=.+,UF.+), ", vplan)
+ m = re.search("graph \[.+(VF=.+,UF.+)", vplan)

It starts to work and I'm able to generate png images just fine. I don't think
there are test cases for this. I'd like to contribute and fix this as my first
issue. Any pointers should test be added for this and where?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20200625/547af771/attachment.html>