This is an archive of the discontinued LLVM Phabricator instance.

[LV] Add a test for register pressure estimation. NFC
Needs ReviewPublic

Authored by yrouban on Apr 4 2023, 8:49 PM.

Details

Summary

The test shows that the interleave count is limited even if there is no local users other than one induction variable.

This test will be changed in the subsequent change D147434.

Diff Detail

Event Timeline

yrouban created this revision.Apr 4 2023, 8:49 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 4 2023, 8:49 PM
Herald added a subscriber: StephenFan. · View Herald Transcript
yrouban requested review of this revision.Apr 4 2023, 8:49 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 4 2023, 8:49 PM
yrouban updated this revision to Diff 510992.Apr 4 2023, 9:00 PM

Updated the test to use opaque pointers as suggested by @fhahn.

fhahn added inline comments.May 1 2023, 5:29 AM
llvm/test/Transforms/LoopVectorize/X86/interleave-count.ll
9

please add a comment indicating what this is testing. Might also e good to have. amore descriptive test file name,

27

is this needed?

36

Is the branch weight needed?

yrouban updated this revision to Diff 520929.May 10 2023, 1:12 AM
yrouban marked 3 inline comments as done.

addressed comments.