[PATCH] D56587: Fix sign/zero extension in Dwarf expressions.
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 7 15:22:47 PST 2019
aprantl added inline comments.
================
Comment at: include/llvm/CodeGen/DIE.h:685
public:
+ MCSymbol *Label = nullptr;
DIE() = delete;
----------------
It seems excessive to burden every DIE object with an extra 8 bytes for something we'll only need on a handful of basic type DIEs. Would it be possible to store this in a DenseMap on the side instead?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56587/new/
https://reviews.llvm.org/D56587
More information about the llvm-commits
mailing list