[PATCH] D12385: Generating Assumption loads fix

Piotr Padlewski via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 26 15:43:34 PDT 2015


Prazek created this revision.
Prazek added reviewers: rsmith, majnemer, rjmccall.
Prazek added a subscriber: cfe-commits.

It wasn't always safe to generate assumption loads. Last time build failed on linking of classes like FenceInst because
it didn't introduce any new virtual function, and it had implicit virtual destructor.

http://reviews.llvm.org/D12385

Files:
  lib/CodeGen/CGCXXABI.h
  lib/CodeGen/CGClass.cpp
  lib/CodeGen/CGVTables.cpp
  lib/CodeGen/ItaniumCXXABI.cpp
  lib/CodeGen/MicrosoftCXXABI.cpp
  test/CodeGenCXX/template-instantiation.cpp
  test/CodeGenCXX/thunks.cpp
  test/CodeGenCXX/vtable-assume-load.cpp
  test/CodeGenCXX/vtable-available-externally.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12385.33262.patch
Type: text/x-patch
Size: 11393 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150826/539d7411/attachment.bin>


More information about the cfe-commits mailing list