[PATCH] D10973: Add missing builtins to the PPC back end for ABI compliance (vol. 3)

hfinkel at anl.gov hfinkel at anl.gov
Fri Jul 10 02:47:30 PDT 2015


hfinkel accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: lib/Target/PowerPC/PPCInstrVSX.td:1175
@@ -1174,1 +1174,3 @@
   }
+  def : Pat<(int_ppc_vsx_xxleqv v4i32:$A, v4i32:$B),
+            (XXLEQV $A, $B)>;
----------------
nemanjai wrote:
> hfinkel wrote:
> > Please put this pattern with the other similar ones (around like 1000 it seems).
> > 
> > 
> I can put it closer to those, but this one needs the definition of XXLEQV and requires HasP8Vector.
We should talk about how to best organize these files, because the scheme that existed previously has now been disrupted somewhat (not obviously in a bad way, but different from what we had before).

Please just move this to after where XXLEQV and XXLNAND are defined for now.

LGTM.



Repository:
  rL LLVM

http://reviews.llvm.org/D10973







More information about the llvm-commits mailing list