This is an archive of the discontinued LLVM Phabricator instance.

[LangRef] insertelement/extractelement return poison for out of range.
ClosedPublic

Authored by efriedma on Jun 6 2018, 4:56 PM.

Details

Summary

We need to clarify the language here. I think poison makes more sense than undef, since it's an undefined operation rather than uninitialized memory. I don't think anything depends on the difference at the moment, though.

Diff Detail

Repository
rL LLVM

Event Timeline

efriedma created this revision.Jun 6 2018, 4:56 PM
nlopes accepted this revision.Jun 7 2018, 6:01 AM

LGTM

This revision is now accepted and ready to land.Jun 7 2018, 6:01 AM
This revision was automatically updated to reflect the committed changes.