[PATCH] D70128: [ThinLTO] Add correctness check for RO/WO variable import

[Eugene Leviant via Phabricator via llvm-commits]

evgeny777 updated this revision to Diff 229040.
evgeny777 added a comment.
Herald added subscribers: dang, steven_wu, mehdi_amini.

Addressed

> This also makes me think it might be a good idea to consider changing the ExportLists to hold the ValueInfos rather than GUIDs

Makes sense, I've done it for this patch as well.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70128/new/

https://reviews.llvm.org/D70128

Files:
  llvm/include/llvm/LTO/LTO.h
  llvm/include/llvm/Transforms/IPO/FunctionImport.h
  llvm/include/llvm/Transforms/IPO/WholeProgramDevirt.h
  llvm/lib/LTO/LTO.cpp
  llvm/lib/LTO/ThinLTOCodeGenerator.cpp
  llvm/lib/Transforms/IPO/FunctionImport.cpp
  llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70128.229040.patch
Type: text/x-patch
Size: 10145 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191113/8dd55c55/attachment.bin>