[llvm-commits] [patch] Fix LegalizeDAG to convert ISD::MUL to libcall is MUL operation is not found.

Mikael Lepistö mikael.lepisto at tut.fi
Sat Oct 4 14:09:31 PDT 2008


Yes, I ran the tests (ran them again just before writing this reply).  
Before patch legalize dag pass just crashed if there was not any mul  
operations. So I thought that it would be ok to emit libcall in place  
where it's found out that we really don't have the operation.

-mikael

		===  Summary ===

# of expected passes		2769
# of expected failures		6

On 4.10.2008, at 23:27, Chris Lattner wrote:

>
> On Oct 3, 2008, at 1:37 AM, Mikael Lepistö wrote:
>
>> Hi,
>>
>> I tried to compile stuff without mul operation in target and  
>> LegalizeDAG failed when it tried to expand ISD::MUL. This patch  
>> seemed to help.
>
> Have you run dejagnu tests with this patch applied?  It appears that  
> it would disable all the mulhs/mulhu optimizations below it.
>
> -Chris






More information about the llvm-commits mailing list