[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