[PATCH] D46018: [GlobalISel][IRTranslator] Split aggregates during IR translation
Roman Tereshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 26 20:41:12 PDT 2018
rtereshin added inline comments.
================
Comment at: include/llvm/CodeGen/GlobalISel/IRTranslator.h:78
+ using const_offset_iterator =
+ DenseMap<const Value *, OffsetListT *>::const_iterator;
+
----------------
This looks like over 128 bytes per `Value` or more. How does memory consumption change with this patch?
If it ends up being a problem, we might reuse the `MachineInstr`s approach with storing machine memory operands. It's a similar pattern - in absolute majority of the cases we have just one to one (in case of value to vregs, or one to zero in case of the machine instruction to memory operands) mapping, but only sometimes its one to many.
Repository:
rL LLVM
https://reviews.llvm.org/D46018
More information about the llvm-commits
mailing list