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

            Bug ID: 38137
           Summary: Inconsistency in printing Windows filespecs
           Product: libraries
           Version: trunk
          Hardware: PC
                OS: Windows NT
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: MC
          Assignee: unassignedb...@nondot.org
          Reporter: paul_robin...@playstation.sony.com
                CC: apra...@apple.com, dblai...@gmail.com,
                    jdevliegh...@apple.com, llvm-bugs@lists.llvm.org

llvm/test/MC/ELF/debug-prefix-map.s was updated in r336793 to test
a case with absolute paths.

The change used the usual {{/|\\}} regex to try to accommodate Windows
path separators, however it still failed:

http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/18383

The relevant detail is:

# command stderr:
C:\ps4-buildslave2\llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast\llvm.src\test\MC\ELF\debug-prefix-map.s:23:13:
error: expected string not found in input

// MAP_ABS: DW_AT_name [DW_FORM_string] ("{{/|\\}}src_root{{(/|\\)}}src.s")
            ^
<stdin>:1:1: note: scanning from here
out.o: file format ELF64-x86-64
^
<stdin>:10:2: note: possible intended match here
 DW_AT_name [DW_FORM_string] ("/src_root\\src.s")
 ^


Note that the backslash is doubled, which the regex can't accommodate.

As a temporary quick fix I added a "+" suffix to the patterns to allow
multiple separators.  However, really we shouldn't be doubling the
backslash.  We need to track down why that's happening and take care of it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to