[PATCH] D51664: [IR] Lazily number instructions for local dominance queries

Sanjoy Das via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 24 11:49:07 PDT 2018


sanjoy added a comment.

> The downside is that Instruction grows from 56 bytes to 64 bytes, and I don't have a good way to measure what that costs in practice.

In the commit that removed the table you said "Removing the virtual table pointer from Value saves 1% of RSS when doing LTO of llc on Linux."; so I'd expect the regression to be in the same ballpark?


https://reviews.llvm.org/D51664





More information about the llvm-commits mailing list