[PATCH] D120057: [mlir][Affine] Fix -lower-affine -debug assertion

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 21 04:01:59 PST 2022


c-rhodes abandoned this revision.
c-rhodes added a comment.

In D120057#3332515 <https://reviews.llvm.org/D120057#3332515>, @sgrechanik wrote:

> We are trying to fix this in a general way by running verification before printing and switching to generic op printing on failure, here: https://reviews.llvm.org/D117834

Thanks for pointing that out, the issue did seem more fundamental and this fix felt like a band-aid, but your patch addresses that. I'll abandon this change.

It might be worth mentioning on the bug report you're working on a fix: https://github.com/llvm/llvm-project/issues/50530


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120057/new/

https://reviews.llvm.org/D120057



More information about the llvm-commits mailing list