[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