[lld] [ELF, test] Add test for R_AARCH64_* implicit addends (PR #87733)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 5 10:51:34 PDT 2024


MaskRay wrote:

> LGTM, but I have a question: will `not` suffice, or do we require `not --crash`? Is `REQUIRES: asserts` also needed?

getImplicitAddend is used by both static relocations and dynamic relocations. The dynamic relocations are only for `config->checkDynamicRelocs` checks, where there is a `#ifdef NDEBUG` difference.

Static relocations don't need `REQUIRES: asserts`.



https://github.com/llvm/llvm-project/pull/87733


More information about the llvm-commits mailing list