This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add tests for cost modelling constants in phis
ClosedPublic

Authored by luke on Apr 25 2023, 10:22 AM.

Diff Detail

Unit TestsFailed

Event Timeline

luke created this revision.Apr 25 2023, 10:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 25 2023, 10:22 AM
luke requested review of this revision.Apr 25 2023, 10:22 AM

Would be good to see tests for other types/vector lengths too

luke updated this revision to Diff 517592.Apr 27 2023, 8:57 AM

Add more tests

ABataev added inline comments.Apr 27 2023, 9:05 AM
llvm/test/Analysis/CostModel/RISCV/phi-const.ll
3

i16, double, i64?

llvm/test/Analysis/CostModel/RISCV/rvv-phi-const.ll
5

Same, tests for i16, i64, double

luke updated this revision to Diff 517627.Apr 27 2023, 10:13 AM

Add more tests for different types

This revision is now accepted and ready to land.Apr 27 2023, 10:18 AM
This revision was landed with ongoing or failed builds.Jun 29 2023, 5:55 AM
This revision was automatically updated to reflect the committed changes.
evandro removed a subscriber: evandro.Jun 30 2023, 3:44 PM