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

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 11 10:36:08 PDT 2017


rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/SIFixSGPRCopies.cpp:344
+
+    Visited.insert(mbb);
+    if (hasTerminatorThatModifiesExec(*mbb, *TRI))
----------------
If you use Visited.insert(mbb).second it is one search in the set instead of two.


Repository:
  rL LLVM

https://reviews.llvm.org/D31350





More information about the llvm-commits mailing list