This is an archive of the discontinued LLVM Phabricator instance.

[Legalizer] Fix wrong operand in split vector helper
ClosedPublic

Authored by qiucf on Jul 8 2020, 2:39 AM.

Details

Summary

It's hard to provide a test case now since vector int-to-fp isn't ready on PowerPC. But this should be easy to review. I guess it's a typo introduced in D69275.

It may cause an unknown segfault in getNode.

Diff Detail

Event Timeline

qiucf created this revision.Jul 8 2020, 2:39 AM
uweigand accepted this revision.Jul 8 2020, 5:00 AM

Yes, this does look like an obvious typo. LGTM!

This revision is now accepted and ready to land.Jul 8 2020, 5:00 AM
This revision was automatically updated to reflect the committed changes.