[PATCH] D74986: [LiveDebugValues] Encode register location within VarLoc IDs [3/3]

Wed May 27 10:50:32 PDT 2020

vsk added a comment.

In D74986#2057531 <https://reviews.llvm.org/D74986#2057531>, @nikic wrote:

> While bisecting compile-time regressions, I hit on this change as a larger -O0 -g regression. It's 2% geomean on CTMark, with the largest regression being 4% on sqlite3. Might be worth a look.

@nikic We actually saw a similar result on our CTMark bot, but the regression was fixed by D76465 <https://reviews.llvm.org/D76465> + D76467 <https://reviews.llvm.org/D76467>. Were you able to measure any improvement after those changes (or, put another way, does reverting D74986 <https://reviews.llvm.org/D74986> + D76465 <https://reviews.llvm.org/D76465> + D76467 <https://reviews.llvm.org/D76467> fix the issue)?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74986