This is an archive of the discontinued LLVM Phabricator instance.

[ISD] Add disclaimer comments to AssertSext/Zext/Align opcodes about poison values
ClosedPublic

Authored by RKSimon on Jul 19 2021, 3:00 AM.

Details

Summary

As encountered on D106053, we need to be very explicit that the Assertion nodes don't hold true for a poison value (or for specific poisoned vector elements).

Diff Detail

Event Timeline

RKSimon requested review of this revision.Jul 19 2021, 3:00 AM
RKSimon created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJul 19 2021, 3:00 AM
efriedma accepted this revision.Jul 19 2021, 8:07 AM
efriedma added a subscriber: efriedma.

LGTM

This revision is now accepted and ready to land.Jul 19 2021, 8:07 AM
This revision was landed with ongoing or failed builds.Jul 19 2021, 9:16 AM
This revision was automatically updated to reflect the committed changes.