[PATCH] D34726: AMDGPU/SI: Do not insert an instruction into worklist twice in movetovalu

[Tony Tye via Phabricator via llvm-commits]

t-tye added a comment.

A concern would be that if using the set caused running the same test multiple times resulted in different results because the different values of pointers caused the iteration order to change. Just because the tests run on your machine does not mean they will run on another machine and get the same result if it is relying on the hashed values of pointers.


https://reviews.llvm.org/D34726