[llvm-dev] Clang for the PlayStation 2

[Dan Ravensloft via llvm-dev]

On Thu, 6 Sep 2018 at 20:01, Tim Northover <t.p.northover at gmail.com> wrote:

> I just did a very quick experiment where I made lowerFP_TO_SINT and
> lowerFP_TO_SINT_STORE return SDValue() (which is the marker for "I
> don't want to handle this").


I just tried this, but the compiler still crashes with the same error.
Maybe our experiments were different.

To make it production-quality you'd want to predicate the changes I
> made on Subtarget->isSingleFloat() I think (probably in combination
> with the actual types, since f32 -> i32 ought to still be OK with the
> existing code). The main annoyance there is that lowerFP_TO_SINT_STORE
> is static rather than a member of MipsISelLowering so it doesn't have
> access to Subtarget. Personally, I'd just make it a member function to
> fix that.
>

lowerFP_TO_SINT_STORE is only ever called by lowerFP_TO_SINT, so I'm just
passing single-floatness (we need a better name for that) as an argument to
lowerFP_TO_SINT_STORE at the moment.

>
> Cheers.
>
> Tim.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20180907/8b44f81e/attachment.html>