This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add another test showing incorrect BUILD_VECTOR lowering
ClosedPublic

Authored by frasercrmck on Apr 19 2022, 1:39 AM.

Details

Summary

This test shows a (contrived) BUILD_VECTOR which is correctly identified
as a sequence of ((vid * -3) / 8) + 5. However, the issue is that using
shift-right for the divide is invalid as the step values are negative.

This patch just adds the test: the fix is added in D123796.

Diff Detail

Event Timeline

frasercrmck created this revision.Apr 19 2022, 1:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 19 2022, 1:39 AM
frasercrmck requested review of this revision.Apr 19 2022, 1:39 AM

fix check for negatives

  • restore correct patch
This revision is now accepted and ready to land.Apr 20 2022, 9:24 PM
This revision was landed with ongoing or failed builds.Apr 20 2022, 11:07 PM
This revision was automatically updated to reflect the committed changes.