Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

Differential D77898

[SVE] Remove calls to getBitWidth from Analysis
ClosedPublic
Actions

Authored by ctetreau on Apr 10 2020, 1:36 PM.

Details

Reviewers
efriedma
sdesmalen
• jnspaulsson
jonpa
Commits
rG56e4888627b3: [SVE] Remove calls to getBitWidth from Analysis

Diff Detail

Event Timeline

ctetreau created this revision.Apr 10 2020, 1:36 PM
Herald added a reviewer: efriedma. · View Herald TranscriptApr 10 2020, 1:36 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added subscribers: llvm-commits, psnobl, rkruppe, tschuett. · View Herald Transcript
ctetreau added a parent revision: D77587: [SVE] Add new VectorType subclasses.Apr 10 2020, 2:02 PM
ctetreau added a child revision: D77833: [SVE] Remove VectorType::getBitWidth().Apr 10 2020, 2:05 PM
Harbormaster failed remote builds in B52725: Diff 256647!Apr 10 2020, 2:32 PM
ctetreau added reviewers: sdesmalen, • jnspaulsson, jonpa.Apr 13 2020, 1:12 PM
ctetreau updated this revision to Diff 257158.Apr 13 2020, 4:55 PM

rebase

ctetreau removed a parent revision: D77587: [SVE] Add new VectorType subclasses.Apr 13 2020, 4:56 PM
Harbormaster failed remote builds in B53036: Diff 257158!Apr 13 2020, 5:32 PM
ctetreau updated this revision to Diff 257811.Apr 15 2020, 1:04 PM

rebase

Harbormaster failed remote builds in B53415: Diff 257811!Apr 15 2020, 1:13 PM
efriedma added inline comments.Apr 15 2020, 1:16 PM
llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
634–635

Fix the comment?

ctetreau updated this revision to Diff 258117.Apr 16 2020, 11:52 AM

fix stale comment

Harbormaster failed remote builds in B53604: Diff 258117!Apr 16 2020, 12:15 PM
efriedma accepted this revision.Apr 16 2020, 3:54 PM

LGTM

This revision is now accepted and ready to land.Apr 16 2020, 3:54 PM
Closed by commit rG56e4888627b3: [SVE] Remove calls to getBitWidth from Analysis (authored by ctetreau). · Explain WhyApr 20 2020, 2:06 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
llvm/
include/
llvm/
Analysis/
6 lines

Diff 258838

View Options

llvm/include/llvm/Analysis/TargetTransformInfoImpl.h

Loading...