diff options
author | Christopher Bate <cbate@nvidia.com> | 2024-12-18 15:22:57 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-12-18 15:22:57 -0700 |
commit | 8272b6bd6146aab973ff7018ad642b99fde00904 (patch) | |
tree | 064ec7d924130bb2ca283b60fb518aa03566fcbd /clang/lib/Frontend/CreateInvocationFromCommandLine.cpp | |
parent | e7a4d78ad328d02bf515b2fa4af8b2c188a6a636 (diff) | |
download | llvm-8272b6bd6146aab973ff7018ad642b99fde00904.zip llvm-8272b6bd6146aab973ff7018ad642b99fde00904.tar.gz llvm-8272b6bd6146aab973ff7018ad642b99fde00904.tar.bz2 |
[mlir][IR] Fix bug in AffineExpr simplifier `lhs % rhs` where `lhs = lhs floordiv rhs` (#119245)
Fixes an issue where the `SimpleAffineExprFlattener` would simplify
`lhs % rhs` to just `-(lhs floordiv rhs)` instead of
`lhs - (lhs floordiv rhs)`
if `lhs` happened to be equal to `lhs floordiv rhs`.
The reported failure case was
`(d0, d1) -> (((d1 - (d1 + 2)) floordiv 8) % 8)`
from https://github.com/llvm/llvm-project/issues/114654.
Note that many paths that simplify AffineMaps (e.g. the AffineApplyOp
folder and canonicalization) would not observe this bug because of
of slightly different paths taken by the code. Slightly different
grouping of the terms could also result in avoiding the bug.
Resolves https://github.com/llvm/llvm-project/issues/114654.
Diffstat (limited to 'clang/lib/Frontend/CreateInvocationFromCommandLine.cpp')
0 files changed, 0 insertions, 0 deletions