diff options
author | Min-Yih Hsu <minyihh@uci.edu> | 2022-04-20 10:40:37 -0700 |
---|---|---|
committer | Min-Yih Hsu <minyihh@uci.edu> | 2022-04-27 09:43:49 -0700 |
commit | a75657d66a1245f78d58c8cb5ded042a13a58299 (patch) | |
tree | 78d34d03bc42e03d0184c602793a793276efad2f /llvm/lib/Transforms/Instrumentation/ControlHeightReduction.cpp | |
parent | ea9bcb8b274ae6ef2fd5cbf8c8c9f42a8aa3c096 (diff) | |
download | llvm-a75657d66a1245f78d58c8cb5ded042a13a58299.zip llvm-a75657d66a1245f78d58c8cb5ded042a13a58299.tar.gz llvm-a75657d66a1245f78d58c8cb5ded042a13a58299.tar.bz2 |
[mlir][LLVMIR] Do not cache llvm::Constant into instMap
Constants in MLIR are not globally unique, unlike that in LLVM IR.
Therefore, reusing previous-translated constants might cause the user
operations not being dominated by the constant (because the
previous-translated ones can be placed in arbitrary place)
This indeed misses some opportunities where we actually can reuse a
previous-translated constants, but verbosity is not our first priority
here.
Differential Revision: https://reviews.llvm.org/D124404
Diffstat (limited to 'llvm/lib/Transforms/Instrumentation/ControlHeightReduction.cpp')
0 files changed, 0 insertions, 0 deletions