Page MenuHomePhabricator

[X86] Fix assertion failure with fp128 to unsigned i64 conversion
ClosedPublic

Authored by mbodart on Oct 15 2015, 12:35 PM.

Details

Summary

My earlier change r245924 introduced an early return and assertion in FP_TO_INTHelper which only considered the f16, f32, f64 and f80 types. Turns out it also needs to return early for fp128, rather than assert.

Diff Detail

Repository
rL LLVM

Event Timeline

mbodart updated this revision to Diff 37505.Oct 15 2015, 12:35 PM
mbodart retitled this revision from to [X86] Fix assertion failure with fp128 to unsigned i64 conversion.
mbodart updated this object.
mbodart added a reviewer: mkuper.
mbodart added a subscriber: llvm-commits.
mkuper accepted this revision.Oct 16 2015, 9:49 AM
mkuper edited edge metadata.

LGTM

This revision is now accepted and ready to land.Oct 16 2015, 9:49 AM
This revision was automatically updated to reflect the committed changes.