This is an archive of the discontinued LLVM Phabricator instance.

[test] Add -inject-tli-mapping to -loop-vectorize -vector-library tests
ClosedPublic

Authored by aeubanks on Aug 25 2020, 11:17 AM.

Details

Summary

The legacy LoopVectorize has a dependency on InjectTLIMappingsLegacy.
That cannot be expressed in the new PM since they are both normal
passes. Explicitly add -inject-tli-mappings as a pass.

Follow-up to https://reviews.llvm.org/D86492.

Diff Detail

Event Timeline

aeubanks created this revision.Aug 25 2020, 11:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 25 2020, 11:17 AM
aeubanks requested review of this revision.Aug 25 2020, 11:17 AM
spatel accepted this revision.Aug 25 2020, 11:48 AM

LGTM

llvm/test/Transforms/LoopVectorize/PowerPC/widened-massv-call.ll
3

Side note: That looks weird. Running the whole -O2 pipeline followed by an extra round of loop-vectorize?

This revision is now accepted and ready to land.Aug 25 2020, 11:48 AM