[Mlir-commits] [mlir] [MLIR] Fuse parent region location when hoisting constants (PR #75258)

Mehdi Amini llvmlistbot at llvm.org
Wed Dec 13 01:21:26 PST 2023


joker-eph wrote:

Why is this different from other CSE cases (or other code motion)?

>  the constant op that is moved (or materialized elsewhere) should have its location fused with the location of the parent region of its insertion point. 

Is this description actually correct? Or is it taking advantage that we always materialize at the beginning of the entry block of a region?

https://github.com/llvm/llvm-project/pull/75258


More information about the Mlir-commits mailing list