[PATCH] D31187: Fix removal of out-of-line definitions.

Richard Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 22 17:17:09 PDT 2017


rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.

The patch itself LGTM. I'd like some test coverage, but if this will be covered by some upcoming interpreter piece and you need this to unblock yourselves, that's good enough for me. In any case, adding a full-blown UnitTest check for this seems like overkill...


https://reviews.llvm.org/D31187





More information about the cfe-commits mailing list