Add comments to remind everyone to keep the get_vtable impls in sync

This commit is contained in:
Oliver Scherer 2018-10-12 17:08:36 +02:00
parent ef3ece7444
commit c95ee9e91d
2 changed files with 8 additions and 0 deletions

View File

@ -94,6 +94,10 @@ pub fn get_vtable(
});
let (size, align) = cx.size_and_align_of(ty);
// /////////////////////////////////////////////////////////////////////////////////////////////
// If you touch this code, be sure to also make the corresponding changes to
// `get_vtable` in rust_mir/interpret/traits.rs
// /////////////////////////////////////////////////////////////////////////////////////////////
let components: Vec<_> = [
callee::get_fn(cx, monomorphize::resolve_drop_in_place(cx.tcx, ty)),
C_usize(cx, size.bytes()),

View File

@ -46,6 +46,10 @@ impl<'a, 'mir, 'tcx, M: Machine<'a, 'mir, 'tcx>> EvalContext<'a, 'mir, 'tcx, M>
let ptr_size = self.pointer_size();
let ptr_align = self.tcx.data_layout.pointer_align;
// /////////////////////////////////////////////////////////////////////////////////////////
// If you touch this code, be sure to also make the corresponding changes to
// `get_vtable` in rust_codegen_llvm/meth.rs
// /////////////////////////////////////////////////////////////////////////////////////////
let vtable = self.memory.allocate(
ptr_size * (3 + methods.len() as u64),
ptr_align,