[PATCH] D31350: AMDGPU : Fix common dominator of two incoming blocks terminates with uniform branch issue.

[Wei Ding via Phabricator via llvm-commits]

wdng added inline comments.


================
Comment at: lib/Target/AMDGPU/SIFixSGPRCopies.cpp:333
+                                 const TargetRegisterInfo *TRI) {
+  std::queue<MachineBasicBlock*> cur, next;
+  cur.push(MBB);
----------------
rampitec wrote:
> You do not need two queues. In fact it is usually done with a SmallVector.
BFS can be implemented in different ways, either using one queue or two. Are there any benefits to use SmallVector? Thanks!


Repository:
  rL LLVM

https://reviews.llvm.org/D31350