[PATCH] D11060: AMDGPU/SI: Select mad patterns to v_mac_f32

Matt Arsenault Matthew.Arsenault at amd.com
Thu Jul 9 17:58:56 PDT 2015


arsenm added inline comments.

================
Comment at: lib/Target/AMDGPU/SIFoldOperands.cpp:131
@@ +130,3 @@
+                              const MachineInstr *MI) {
+  for (auto Candidate : FoldList) {
+    if (Candidate.UseMI == MI)
----------------
tstellarAMD wrote:
> arsenm wrote:
> > FoldList seems like it should be a set or map instead
> A map with UseMI as the Key and FoldCandidate as the value?
Yes. Is there a reason FoldList will be known to be small?


http://reviews.llvm.org/D11060







More information about the llvm-commits mailing list