mmha wrote: My understanding is that we are not supposed to erase an op that was replaced anymore. `eraseOp` calls `replaceOp` internally which lead to the double replacement assertion failure.
I also think in our case it doesn't really matter whether we use `replaceOp` and `replaceAllUseWith` https://github.com/llvm/llvm-project/pull/157471 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
