diff options
| author | Matthias Springer <me@m-sp.org> | 2024-08-17 09:43:30 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-08-17 09:43:30 +0200 |
| commit | cb7614e839148196c53711fdee639e6dff933a8d (patch) | |
| tree | 6afbb74bf838abea2ae8b7ba691aa8e1326d0482 /llvm/lib/CodeGen/CodeGenPrepare.cpp | |
| parent | e6ceb29ab6bb5632cac79e99da4d248e9bb7d378 (diff) | |
| download | llvm-cb7614e839148196c53711fdee639e6dff933a8d.zip llvm-cb7614e839148196c53711fdee639e6dff933a8d.tar.gz llvm-cb7614e839148196c53711fdee639e6dff933a8d.tar.bz2 | |
[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));
```
Diffstat (limited to 'llvm/lib/CodeGen/CodeGenPrepare.cpp')
0 files changed, 0 insertions, 0 deletions