This patch changes the code to use a modern unwrapOrError(StringRef Input, Expected<T> EO)
version that contains the input source name and removes the depricated version.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
test/Object/invalid.test | ||
---|---|---|
229 ↗ | (On Diff #214129) | Aside: it's rather weird that invalid-e_shnum is being used to test an invalid e_phentsize...! |
247 ↗ | (On Diff #214129) | I think you can use -D here, right, like in %t10 case above and others? |
test/tools/yaml2obj/elf-sht-symtab-shndx.yaml | ||
6 ↗ | (On Diff #214129) | Nit: use single-dash for -D. |
74 ↗ | (On Diff #214129) | Ditto |
test/Object/invalid.test | ||
---|---|---|
429 ↗ | (On Diff #214129) | Why can't this check use [[FILE]]? |
- Addressed review comments.
test/Object/invalid.test | ||
---|---|---|
229 ↗ | (On Diff #214129) | Initially this file was introduced to report "e_shnum should be zero if a file has no section header table" (D25090) This is a result of error catching code change multiplied by effect from having a black box precompiled binary committed. |
247 ↗ | (On Diff #214129) | Yes, done. |
429 ↗ | (On Diff #214129) | It can. I did not like how -DFILE=%p/Inputs/corrupt-invalid-phentsize.elf.x86-64 looks like (too long), |