Displaying 2 results from an estimated 2 matches for "4867836".
2018 Sep 25
2
RFC Storing BB order in llvm::Instruction for faster local dominance
...suggested that I re-run the max RSS during
full LTO of llc experiment that I did when I removed the vptr from Value. I
went ahead and did that on the review, but it's not on the dev list, so it
may not have enough visibility. I found that the max RSS before the patch
was 4816464 kb, and after, 4867836 kb. That's an increase of 1.06%, or in
absolute terms, ~50MB of memory spent on instruction positions during full
LTO of llc.
I think Sanjoy's bit-stealing suggestion is a worthwhile alternative to
growing Instruction, but I really think we should analyze it from the
position of, is the co...
2018 Sep 25
2
RFC Storing BB order in llvm::Instruction for faster local dominance
> On Sep 24, 2018, at 10:19 AM, Reid Kleckner <rnk at google.com> wrote:
>
> To echo what Hal said, yes, it's a major change, but I think the improved complexity guarantees, simplicity, and elimination of certain classes of bugs is worth it.
>
> I think we have consensus that we should go forward with this. Would anyone mind formally stamping it in phab? So far everyone