[PATCH] D30716: Fix one-after-the-end type metadata handling in globalsplit

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 7 13:55:42 PST 2017


pcc added a comment.

LGTM

This is a bit of a hack because we're assuming that vtable groups are the only kinds of global variables that !type metadata can be attached to, and that they are either Itanium ABI vtable groups or contain a single vtable (i.e. Microsoft ABI vtables). To avoid needing to make that assumption we may want to represent the type metadata offsets as something like a set of GEP indices instead of a byte offset. But this is fine for now. Please leave a comment stating that we're making this assumption.


Repository:
  rL LLVM

https://reviews.llvm.org/D30716





More information about the llvm-commits mailing list