[PATCH] D43690: [ThinLTO] Keep available_externally symbols live

[Taewook Oh via Phabricator via llvm-commits]

twoh added a comment.
Herald added subscribers: dexonsmith, steven_wu.

Hello, I wonder if we need to keep linkonce_odr symbols live here as well. I observe a case that a vtable for template class initiated has linkonce_odr linkage and marked dead here, which results compiler crash at WholeProgramDevirt because the global variable for vtable doesn't have initializer (https://github.com/llvm-mirror/llvm/blob/master/lib/Transforms/IPO/WholeProgramDevirt.cpp#L676 assumes that GV has the initializer). Thanks!


Repository:
  rL LLVM

https://reviews.llvm.org/D43690