[PATCH] D95513: [llvm-objdump][test] Fix --prefix tests for system-windows
Vinicius Tinti via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 27 04:12:04 PST 2021
tinti added inline comments.
================
Comment at: llvm/test/tools/llvm-objdump/X86/source-interleave-prefix-windows.test:1
+;; Test --prefix option platform specific behavior.
+; REQUIRES: system-windows
----------------
jhenderson wrote:
> Could you expand this comment to explain what the platform-specific behaviour is, please? (I'm guessing it's related to the trailing separator comment below).
I think I can and you are right.
Take a look at this file llvm/test/tools/llvm-objdump/X86/Inputs/source-interleave.ll
```
...
!0 = distinct !DICompileUnit(language: DW_LANG_C99, file: !1, producer: "clang version 4.0.0", isOptimized: false, runtimeVersion: 0, emissionKind: FullDebug, enums: !2, globals: !3)
!1 = !DIFile(filename: "source-interleave-x86_64.c", directory: "SRC_COMPDIR")
!2 = !{}
...
```
There is a directory entry and a file entry. SRC_COMPDIR will be replaced.
Merging "./Inputs" and "source-interleave-x86_64.c" will produce different results on platforms with different default separators.
Should I mention it on the code and/or commit message? Should I find where they are being merged (llc or llvm-objdump)?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95513/new/
https://reviews.llvm.org/D95513
More information about the llvm-commits
mailing list