aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/ExecutionEngine/Orc/ObjectLinkingLayer.cpp
diff options
context:
space:
mode:
authorRiver Riddle <riddleriver@gmail.com>2021-10-27 02:00:10 +0000
committerRiver Riddle <riddleriver@gmail.com>2021-10-27 02:09:04 +0000
commit015192c63415ae828c3a9bc8b56d0667abcca8ae (patch)
treeb48e0408e5a5174a71738420cd5815725f31917f /llvm/lib/ExecutionEngine/Orc/ObjectLinkingLayer.cpp
parent01b55f163a40073914bf92f5b710319e7cae2e02 (diff)
downloadllvm-015192c63415ae828c3a9bc8b56d0667abcca8ae.zip
llvm-015192c63415ae828c3a9bc8b56d0667abcca8ae.tar.gz
llvm-015192c63415ae828c3a9bc8b56d0667abcca8ae.tar.bz2
[mlir:DialectConversion] Restructure how argument/target materializations get invoked
The current implementation invokes materializations whenever an input operand does not have a mapping for the desired type, i.e. it requires materialization at the earliest possible point. This conflicts with goal of dialect conversion (and also the current documentation) which states that a materialization is only required if the materialization is supposed to persist after the conversion process has finished. This revision refactors this such that whenever a target materialization "might" be necessary, we insert an unrealized_conversion_cast to act as a temporary materialization. This allows for deferring the invocation of the user materialization hooks until the end of the conversion process, where we actually have a better sense if it's actually necessary. This has several benefits: * In some cases a target materialization hook is no longer necessary When performing a full conversion, there are some situations where a temporary materialization is necessary. Moving forward, these users won't need to provide any target materializations, as the temporary materializations do not require the user to provide materialization hooks. * getRemappedValue can now handle values that haven't been converted yet Before this commit, it wasn't well supported to get the remapped value of a value that hadn't been converted yet (making it difficult/impossible to convert multiple operations in many situations). This commit updates getRemappedValue to properly handle this case by inserting temporary materializations when necessary. Another code-health related benefit is that with this change we can move a majority of the complexity related to materializations to the end of the conversion process, instead of handling adhoc while conversion is happening. Differential Revision: https://reviews.llvm.org/D111620
Diffstat (limited to 'llvm/lib/ExecutionEngine/Orc/ObjectLinkingLayer.cpp')
0 files changed, 0 insertions, 0 deletions