[PATCH] D39396: Fix for PR33930. Short-circuit metadata mapping when cloning a varargs thunk.

Wolfgang Pieb via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 30 18:23:48 PDT 2017


wolfgangp marked an inline comment as done.
wolfgangp added a comment.

In https://reviews.llvm.org/D39396#911306, @aprantl wrote:

> This works for me, but as I said previously, perhaps we can get by with just not having any variables described in the thunk to further simplify the code.


I remember, but this "thunk" isn't really a thunk in the sense that it makes some adjustments and then branches to (or calls) the real function. It effectively becomes the real function (hence the cloning), so if you drop the variables you lose the ability to debug it.


https://reviews.llvm.org/D39396





More information about the cfe-commits mailing list