[PATCH] First stage of call lowering for Mips fast-isel

Eric Christopher echristo at gmail.com
Wed Oct 15 17:40:13 PDT 2014


There's no testcase showing up in this diff at all FWIW.

================
Comment at: lib/Target/Mips/MipsFastISel.cpp:183
@@ +182,3 @@
+
+bool (*UnusedMipsFastISel1)(unsigned int, llvm::MVT, llvm::MVT,
+                            CCValAssign::LocInfo, ISD::ArgFlagsTy,
----------------
rkotler wrote:
> echristo wrote:
> > Eh? What's with all of these?
> I have a patch that fixes that with function attributes as in:
> 
>     static bool CC_Mips16RetHelper(unsigned ValNo, MVT ValVT,
>                        MVT LocVT, CCValAssign::LocInfo LocInfo,
>                        ISD::ArgFlagsTy ArgFlags, CCState &State)
>     __attribute__((unused));
> 
> 
Best to just leave them out for now.

http://reviews.llvm.org/D5714






More information about the llvm-commits mailing list