[PATCH] D64630: [DebugInfo] Address performance regression with r364515
Jeremy Morse via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 8 11:21:30 PST 2019
jmorse added a comment.
Vedant wrote:
> I guess there are a few alternatives to consider that call for changing data structures. Perhaps it makes more sense to start with the approach taken here to address the performance issue and to keep an eye out for any more problems.
Indeed, I'd much prefer to design it out; the current scenario isn't ideal for a number of reasons.
> BTW @jmorse is this patch still rebased on top of r364515? I don't see mergingChangesDbgValue anymore.
*blinks* ah yeah, the latest update should fix that.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64630/new/
https://reviews.llvm.org/D64630
More information about the llvm-commits
mailing list