[Mlir-commits] [mlir] Revert "[MLIR] Fuse locations of merged constants (#74670)" (PR #75381)

Fangrui Song llvmlistbot at llvm.org
Wed Dec 13 13:38:21 PST 2023


MaskRay wrote:

> If this is causing significant issues with OOM / timeout, I'm open to reverting for now and finding a more efficient approach (I'm proposing one in #75258).
> 
> Do you have any data that suggests why the previous fix (#75218) didn't solve the problem? Was it generating super long fused locations? Or was it just due to all the temporary fused locations that were created since we "append" locations incrementally?

Thank you for being open for reverts! I don't know the technical aspect of the issue... but I'll find right folks help investigate this issue. Stay tuned!

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


More information about the Mlir-commits mailing list