[PATCH] D127581: [ELF] Relax R_RISCV_ALIGN
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 19 21:39:42 PDT 2022
MaskRay added a comment.
In D127581#3594812 <https://reviews.llvm.org/D127581#3594812>, @tschuett wrote:
> Do you really need a plain union? How about a `PointerUnion` or something llvmy `std::variant`?
The size is very important for memory usage. Since `PointerUnion` abstraction seems unnecessary, I'd want to avoid it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127581/new/
https://reviews.llvm.org/D127581
More information about the llvm-commits
mailing list