[PATCH] D53986: [lto] Fix a crash caused by accessing an empty ValueInfo

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 2 09:40:01 PDT 2018


tejohnson added a comment.

The change looks ok to me now, but we really need a test case for this. I tried unsuccessfully so far to induce the error. Was this produced when using the save-temps plugin-opt during your ThinLTO link? Can you go into the debugger and see what the value of E.Dst is?

I'm also at Google so feel free to reach out to me internally if your reproducer can't be shared upstream.


Repository:
  rL LLVM

https://reviews.llvm.org/D53986





More information about the llvm-commits mailing list