[PATCH] D68816: [NFC] Replace a linked list in LiveDebugVariables pass with a DenseMap

Orlando Cazalet-Hyams via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 16 01:45:35 PDT 2019


Orlando added a comment.

After timing some more self-hosts it looks like my original numbers which looked
too good to be true sadly are just that. The self-host asan build time reduction 
seems correct (-3.5%) but the non-asan builds are much less impacted then first
thought (about -1%). These numbers are more in line with my original expectation.

After doing some more self-host builds and building a smaller llvm-based project
16 times it seems like the SmallDenseMap makes essentially no difference.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68816/new/

https://reviews.llvm.org/D68816





More information about the llvm-commits mailing list