Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
- Build Status
Buildable 45464 Build 47332: arc lint + arc unit
Event Timeline
I left similar comments on D72357 too
lld/test/COFF/start-lib.ll | ||
---|---|---|
26 | I'm not sure what this is supposed to be. Can it just be Name: bar? Ditto for TEST3-NOT below | |
llvm/test/tools/llvm-objcopy/ELF/only-section-strip-undefined.test | ||
6 ↗ | (On Diff #236673) | Read from stdin to avoid the filename showing up, e.g. # RUN: llvm-readobj --symbols < %t2.o | FileCheck %s --implicit-check-not=bar Then the rest of the test can remain unchanged |
llvm/test/tools/llvm-objcopy/ELF/tail-merged-string-tables.test | ||
7–8 ↗ | (On Diff #236673) | Same here, read from stdin |
lld/test/COFF/start-lib.ll | ||
---|---|---|
35–36 | Is {{ }}foo good enough? The map file is something like 00001010 00000000 0 bar |
llvm/test/tools/llvm-objcopy/ELF/only-section-strip-undefined.test | ||
---|---|---|
6 ↗ | (On Diff #236673) | This will collide with D72357. As noted in that review, I'd prefer just changing the patterns to "Name: bar" (e.g. --implicit-check-not="Name: bar"). |
llvm/test/tools/llvm-objcopy/ELF/tail-merged-string-tables.test | ||
7–8 ↗ | (On Diff #236673) | See my above comment/comment in D72357. |
lld/test/COFF/start-lib.ll | ||
---|---|---|
26 | Name: bar doesn’t work – the map file contains 00001010 00000000 0 bar However, @MaskRay’s suggestion below applies here as well. Changed to {{ }}bar. | |
35–36 | Yes, that works great. Changed. | |
llvm/test/tools/llvm-objcopy/ELF/only-section-strip-undefined.test | ||
6 ↗ | (On Diff #236673) | Obviated by D72358. |
llvm/test/tools/llvm-objcopy/ELF/tail-merged-string-tables.test | ||
7–8 ↗ | (On Diff #236673) | Also obviated by D72358. |
lld/test/COFF/start-lib.ll | ||
---|---|---|
26 | If that's the format, then the TEST2-NOT lines either side of this line won't work... Also, perhaps consider {{ }}bar{{$}, since {{ }}bar could match somebody with a space in their path before "bar". |
Check for end-of-line in start-lib.ll, and fix assertion issue
lld/test/COFF/start-lib.ll | ||
---|---|---|
26 | Ah, good point. Switched everywhere to {{ }}symbolName{{$}}. |
lld/test/COFF/start-lib.ll | ||
---|---|---|
35–36 | Possibly want to change these lines too to have {{$}} at the end? |
lld/test/COFF/start-lib.ll | ||
---|---|---|
26 | It may be helpful to add ; TEST2: Address Size Align Out In Symbol before the first check line. The sames applies to TEST3 |
I'm not sure what this is supposed to be. Can it just be Name: bar? Ditto for TEST3-NOT below