[PATCH] Use GEPs correctly when adjusting "this" in MicrosoftCXXABI

Chandler Carruth chandlerc at google.com
Tue Oct 22 17:59:26 PDT 2013


On Fri, Oct 18, 2013 at 8:46 AM, Timur Iskhodzhanov <timurrrr at google.com>wrote:

> I'll fix the GEP UB in the thunk emission code as part of my next vtordisp
> patch.


FYI, I would love to see more incremental patches here. If you have a
bugfix to code you're already working on a new feature in, I think it is
often a worthwhile tradeoff to fix the bug, and then merge that fix into
your existing work. Not sure if that's the case here, just wanted to
mention it from the peanut gallery.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131022/e5584c11/attachment.html>


More information about the cfe-commits mailing list