[PATCH] D104425: [AMDGPU] Fixed constexpr expansion to handle multiple uses

[Artem Belevich via Phabricator via llvm-commits]

tra accepted this revision.
tra added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/IR/ReplaceConstant.cpp:87
     SmallPtrSetImpl<Instruction *> *Insts) {
+  SmallPtrSet<ConstantExpr *, 8> Visited;
   for (Use &U : I->operands()) {
----------------
Do we need/want to bump the initial set size? I'd assume that it would need to be larger now that we keep track of all operands.


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

https://reviews.llvm.org/D104425