llvm-project
cb7614e8 - [mlir][Transforms] Dialect conversion: Fix bug in `computeNecessaryMaterializations` (#104630)

Commit
1 year ago
[mlir][Transforms] Dialect conversion: Fix bug in `computeNecessaryMaterializations` (#104630) There was a typo in the code path that removes unnecessary materializations. Before: Update `opResult` (result of an op different from `user`) in mapping and remove `user`. ``` replaceMaterialization(rewriterImpl, opResult, inputOperands, inverseMapping); necessaryMaterializations.remove(materializationOps.lookup(user)); ``` After: Update `user->getResults()` in mapping and remove `user`. ``` replaceMaterialization(rewriterImpl, user->getResults(), inputOperands, inverseMapping); necessaryMaterializations.remove(materializationOps.lookup(user)); ```
Parents
Loading