[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