Re: [PATCH] sparse, llvm: Simplify comparison op code generation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 22, 2011 at 10:18 AM, Pekka Enberg <penberg@xxxxxxxxxx> wrote:
>> I think that might be a generic sparse floating point issue but I
>> didn't look too closely.

On Tue, Nov 22, 2011 at 8:26 PM, Linus Torvalds
<torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:
> No, it looks like sparse is doing fine, but because you don't generate
> the FP versions of the comparison ops, LLVM is unhappy with getting an
> integer comparison for a floating point value.
>
> Enhancing the translate_op() function to generate the proper FP
> comparisons for LLVM would probably make it "JustWork(tm)".

Unfortunately LLVM API requires LLVMBuildFCmp() for floats so it's not
as simple as that. I sent an incremental patch as a separate email
that adds FP support for comparison ops.
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Newbies FAQ]     [LKML]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Trinity Fuzzer Tool]

  Powered by Linux