[PATCH] D111587: re-land: [clang] Fix absolute file paths with -fdebug-prefix-map

Keith Smiley via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 15 09:19:01 PDT 2022


keith added inline comments.


================
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: "")
----------------
probinson wrote:
> Does this want to be 
> `"%{fs-src-root}OVERRIDE%{fs-sep}DebugObjC.h"` ?
That can only be used in the RUN invocations, but unfortunately regardless it could be used here because of the escaped backslashes, so it requires a regex like this instead 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111587



More information about the cfe-commits mailing list