Page MenuHomePhabricator

[RISCV] Remove ForceTailAgnostic flag from vmv.s.x, vfmv.s.f and reductions.
ClosedPublic

Authored by craig.topper on Jun 4 2021, 9:33 PM.

Details

Summary

In 0.9 these were defined to leave elements other than 0 in the
destination unmodified. They were changed to use the tail policy
in 0.10. I missed that update.

I assume no one has noticed because in order cores treat tail
agnostic the same as tail undisturbed. I believe Spike and QEMU do
the same.

Diff Detail

Event Timeline

craig.topper created this revision.Jun 4 2021, 9:33 PM
craig.topper requested review of this revision.Jun 4 2021, 9:33 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 4 2021, 9:33 PM
Herald added a subscriber: MaskRay. · View Herald Transcript
arcbbb accepted this revision.Jun 8 2021, 1:49 AM

LGTM

This revision is now accepted and ready to land.Jun 8 2021, 1:49 AM
frasercrmck accepted this revision.Jun 8 2021, 1:58 AM

LGTM, nice catch.