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

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/SIFixSGPRCopies.cpp:333
+                                 const TargetRegisterInfo *TRI) {
+  std::queue<MachineBasicBlock*> cur, next;
+  cur.push(MBB);
----------------
wdng wrote:
> 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!
You definitely do not need two. Then llvm preferrs to use its own containers over std.


Repository:
  rL LLVM

https://reviews.llvm.org/D31350