[PATCH] D20636: PR26055: Speed up LiveDebugValues::transferDebugValue()

Adrian Prantl via llvm-commits llvm-commits at lists.llvm.org
Thu May 26 09:47:59 PDT 2016


aprantl added a comment.

In http://reviews.llvm.org/D20636#440225, @dblaikie wrote:

> Given this is a time/space tradeoff, it might be worth having the space
>  numbers as well just to demonstrate that it's a good tradeoff?


This adds one DenseMap entry (16 bytes index + 4 bytes payload) per local (inlined) variable in a single function. In my large benchmark with the >4000 DBG_VALUEs this was below the noise threshold. Peak memory usage seems to fluctuate by ~ 4MB on repeat runs on my machine.


Repository:
  rL LLVM

http://reviews.llvm.org/D20636





More information about the llvm-commits mailing list