Page MenuHomePhabricator
Differential D142674

[LV] Move collect* calls to LVP::plan() (NFC).
Needs ReviewPublic
Actions

Authored by fhahn on Jan 26 2023, 3:07 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Move calls of collect* helpers closer to where the cost-model is used.
Should help simplifying D142669 & D142670.

Diff Detail

Unit TestsFailed
View All

TimeTest
50 msx64 debian > LLVM.Transforms/SCCP::add-nuw-nsw-flags.ll
Script: -- : 'RUN: at line 2'; /var/lib/buildkite-agent/builds/llvm-project/build/bin/opt -passes=ipsccp -S /var/lib/buildkite-agent/builds/llvm-project/llvm/test/Transforms/SCCP/add-nuw-nsw-flags.ll | /var/lib/buildkite-agent/builds/llvm-project/build/bin/FileCheck /var/lib/buildkite-agent/builds/llvm-project/llvm/test/Transforms/SCCP/add-nuw-nsw-flags.ll

Event Timeline

fhahn created this revision.Jan 26 2023, 3:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 26 2023, 3:07 PM
Herald added subscribers: StephenFan, hiraditya. · View Herald Transcript
fhahn requested review of this revision.Jan 26 2023, 3:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 26 2023, 3:07 PM
Herald added subscribers: llvm-commits, pcwang-thead. · View Herald Transcript
fhahn added a child revision: D142669: [VPlan] Allow planning with different cost models (WIP)..Jan 26 2023, 3:07 PM
Harbormaster completed remote builds in B210231: Diff 492582.Jan 26 2023, 4:24 PM

Revision Contents
Changeset List

PathSize
llvm/
lib/
Transforms/
Vectorize/
6 lines

Diff 492582

View Options

llvm/lib/Transforms/Vectorize/LoopVectorize.cpp

Loading...