[all-commits] [llvm/llvm-project] 67db34: [mlir][Transforms][NFC] Dialect Conversion: Earlie...
Matthias Springer via All-commits
all-commits at lists.llvm.org
Sat Jul 12 07:47:03 PDT 2025
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 67db34863cefb4ddb71babbe22416c89cf743564
https://github.com/llvm/llvm-project/commit/67db34863cefb4ddb71babbe22416c89cf743564
Author: Matthias Springer <me at m-sp.org>
Date: 2025-07-12 (Sat, 12 Jul 2025)
Changed paths:
M mlir/lib/Transforms/Utils/DialectConversion.cpp
Log Message:
-----------
[mlir][Transforms][NFC] Dialect Conversion: Earlier `isIgnored` check (#148360)
When legalizing an operation, the conversion driver skips "ignored" ops.
Ops are ignored if they are inside of a recursively legal operation or
if they were erased.
This commit moves the "is ignored" check a bit earlier: it is now
checked before checking if the op is recursively legal. This is in
preparation of the One-Shot Dialect Conversion refactoring: erased ops
should not be accessed, not even for checking recursive legality.
This commit is NFC: When an op is erased, it is added to the set of
ignored ops and we don't want to process it, regardless of legality.
Nested ops are also added to the set of ignored ops when erasing an
enclosing op.
To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications
More information about the All-commits
mailing list