[PATCH] D24927: Propagate DBG_VALUE entries when there are unvisited predecessors

Adrian Prantl via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 09:21:00 PDT 2016


aprantl added a comment.

Thanks! Please watch the compile-time performance bots after committing this. This could cause many more DBG_VALUE instructions to be inserted and could cause a regression.  Contrary to the old LiveDebugVariables pass we currently don't look at the lexical scopes before inserting a DBG_VALUE intrinsic which means that we often propagate them much further down than necessary. This is especially noticeable in large C++ functions with many inlined method calls that all use the same "this"-pointer. I mean to fix this soon but haven't gotten around to it yet.


https://reviews.llvm.org/D24927





More information about the llvm-commits mailing list