These tests were checking for absolute paths and could fail if it included any valid path character that was not included in that regex. This patch avoids the issue by removing the path checks.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
test/elf/dynamic-undef.test | ||
---|---|---|
30 ↗ | (On Diff #18048) | I don't fully understand why now you need to split this into two lines. If "Undefined symbol" is not on the same line as "shared.so-x86-64: puts", why the previous test succeeded? |
test/elf/dynamic-undef.test | ||
---|---|---|
30 ↗ | (On Diff #18048) | I suspect FileCheck looks for "no more than one newline". How about {{.+[\\/]}} instead? |
test/elf/dynamic-undef.test | ||
---|---|---|
30 ↗ | (On Diff #18048) | Maybe EXEC: Undefined symbol: {{.*}}/shared.so-x86-64: puts is enough? |
Comment Actions
Attempt 2: Use a regex that accepts any set of characters that ends in a [back]slash.
test/elf/dynamic-undef.test | ||
---|---|---|
30 ↗ | (On Diff #18048) | Would that work if cross-compiling on Windows? |