[PATCH] D129936: [JITLink][COFF][x86_64] Reimplement ADDR32NB/REL32.

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 16 13:19:58 PDT 2022


tschuett added a comment.

You said for both relocations that the value should not exceed 32bit. Is there a place for asserts to check/prove the fact?

I prefer `enum class` for new code.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129936/new/

https://reviews.llvm.org/D129936



More information about the llvm-commits mailing list