Page MenuHomePhabricator

[Orc] make getResponsibilitySetWithLegacyFn behavior match with LegacyJITSymbolResolver::getResponsibilitySet
Needs ReviewPublic

Authored by twoh on Oct 30 2018, 4:45 PM.

Details

Reviewers
lhames
Summary

https://reviews.llvm.org/rL340874 replaces lookupFlags with getResponsibilitySet, and this introduces a behaviral change in LegacyJITSymbolResolver::getResponsibilitySet (If there is no existing definition then the caller is responsible for materializing it. https://github.com/llvm-mirror/llvm/blob/master/lib/ExecutionEngine/RuntimeDyld/JITSymbol.cpp#L127). However, this changes is missed from getResponsibilitySetWithLegacyFn. Though I'm not very familiar with Orc code base, I think the behavior of these two functions supposed to match, and the patch actually fixes "symbols not found" error in our codebase using OrcJIT.

Diff Detail

Event Timeline

twoh created this revision.Oct 30 2018, 4:45 PM
twoh added a comment.Nov 5 2018, 8:59 AM

Friendly ping. Thanks!