[PATCH] ARM: Implement big endian bit-conversion for NEON types
Christian Pirker
cpirker at a-bix.com
Wed May 7 10:34:39 PDT 2014
Hi James,
> Whereabouts, exactly? I see no changes to the calling convention .td or the calling convention parts of ISel lowering.
These conversions are done implicitly when fitting a vector value into a GPR(s), therefore no changes in the calling convetions .td.
Thanks,
Christian
================
Comment at: lib/Target/ARM/ARMFastISel.cpp:192
@@ -191,1 +191,3 @@
+ const TargetLowering* getTargetLowering() { return TM.getTargetLowering(); }
+
----------------
James Molloy wrote:
> Christian Pirker wrote:
> > James Molloy wrote:
> > > This function seems unused?
> > This function is required to evaluate the [IsBE] predicate for the bitconversion rules that specify the VREV instructions.
> But that predicate isn't defined anywhere in this patch.
The [IsBE] predicate is defined in ARMInstrInfo.td (see line 305).
http://reviews.llvm.org/D3651
More information about the llvm-commits
mailing list