This is an archive of the discontinued LLVM Phabricator instance.

Integer legalization: fix MUL expansion
ClosedPublic

Authored by chfast on Nov 14 2016, 12:20 PM.

Details

Summary

This fixes the runtime results produces by the fallback multiplication expansion introduced in r270720.

For tests I created a fuzz tester that compares the results with Boost.Multiprecision.

Diff Detail

Repository
rL LLVM

Event Timeline

chfast updated this revision to Diff 77860.Nov 14 2016, 12:20 PM
chfast retitled this revision from to Integer legalization: fix MUL expansion.
chfast updated this object.
chfast added a reviewer: hfinkel.
chfast added a subscriber: llvm-commits.
hfinkel accepted this revision.Nov 15 2016, 7:58 AM
hfinkel edited edge metadata.

Ping.

LGTM, thanks!

This revision is now accepted and ready to land.Nov 15 2016, 7:58 AM
This revision was automatically updated to reflect the committed changes.
llvm/trunk/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp