Page MenuHomePhabricator

Fix 2 LLVM assertion caoused by an API missues in Scalar
AbandonedPublic

Authored by tberghammer on Apr 26 2016, 8:53 AM.

Details

Reviewers
clayborg
labath
Summary

Fix 2 LLVM assertion caused by an API miss-ues in Scalar

APInt::trunc only works if the destination size is strictly smaller
then the source size while APInt::sext and APInt::zext only works if
the destination size is strictly larger then the source size.

Diff Detail

Event Timeline

tberghammer retitled this revision from to Fix 2 LLVM assertion caoused by an API missues in Scalar.
tberghammer updated this object.
tberghammer added reviewers: labath, clayborg.
tberghammer added a subscriber: lldb-commits.
labath edited edge metadata.Apr 26 2016, 8:56 AM

Could you please add one or two unit tests for this in ScalarTest?

tberghammer abandoned this revision.May 20 2016, 4:09 AM

Abandoning. as rL270062 fixed the same issue before I managed to add a test for this CL.