probinson added a comment.

In D111587#3381369 <https://reviews.llvm.org/D111587#3381369>, @keith wrote:

> Fix tests with dwarf 6

Do you mean dwarf 5 here?  There is no v6 yet.



================
Comment at: clang/test/Modules/module-debuginfo-prefix.m:24
 
-// Dir should always be empty, but on Windows we can't recognize /var
-// as being an absolute path.
-// CHECK: !DIFile(filename: "/OVERRIDE/DebugObjC.h", directory: 
"{{()|(.*:.*)}}")
+// CHECK: !DIFile(filename: "{{/|.:\\\\}}OVERRIDE{{/|\\\\}}DebugObjC.h", 
directory: "")
----------------
Does this want to be 
`"%{fs-src-root}OVERRIDE%{fs-sep}DebugObjC.h"` ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111587

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to