[PATCH] D103016: NewGVN: Relax assertion about number of times a value is seen

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 13:39:33 PST 2023


arsenm added a comment.

In D103016#4074721 <https://reviews.llvm.org/D103016#4074721>, @kmitropoulou wrote:

> I cannot reproduce the bug. I had to decrease the counter down to 6 in order to activate the assertion. This is normal. The counter is updated after value numbering. An instruction will be processed again if one of its operands is added in a different congruence class.  Do you remember which was the problem?

I was able to reproduce this with the testcase attached to the issue a few weeks ago, I don’t remeymore than that


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

https://reviews.llvm.org/D103016



More information about the llvm-commits mailing list