This is an archive of the discontinued LLVM Phabricator instance.

[VFDatabase] Testsuite for scalar functions are vector functions with VF =1
ClosedPublic

Authored by masoud.ataei on Apr 29 2020, 1:38 PM.

Details

Summary

Fixing test suite of the committed PR: https://reviews.llvm.org/D78054.
I am proposing to remove the PowerPC target triple in the test suite.

Diff Detail

Event Timeline

masoud.ataei created this revision.Apr 29 2020, 1:38 PM

Just minor comments.

Thank you.

llvm/test/Transforms/LoopVectorize/vectorizeVFone.ll
26–27

You might want to remove this too?

27–28

Also, given that all references to ppc seems to be dropped, maybe rename __atand2_massv to vector_atan, so you can simplify the CHECK-NOT: to check for call fast <{{[0-9]+}} x double> @{{.*}}atan(.

masoud.ataei edited the summary of this revision. (Show Details)

Removing other dependencies to PowerPC machines in the test suits.

jsji accepted this revision.Apr 30 2020, 7:48 AM

LGTM. Thanks for fixing.

This revision is now accepted and ready to land.Apr 30 2020, 7:48 AM
fpetrogalli accepted this revision.Apr 30 2020, 7:58 AM

Looks good to me too! Ship it! :)

Thank you,

Francesco

This revision was automatically updated to reflect the committed changes.