Page MenuHomePhabricator

[llvm-exegesis] Explore LEA addressing modes.
ClosedPublic

Authored by courbet on Oct 8 2019, 7:40 AM.

Details

Summary

This will help for PR32326.

This shows the well-known issue with RBP and R13 as base registers.

Diff Detail

Event Timeline

courbet created this revision.Oct 8 2019, 7:40 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 8 2019, 7:40 AM
Herald added a subscriber: tschuett. · View Herald Transcript

Example results for uops on HSW:

gchatelet accepted this revision.Oct 9 2019, 1:08 AM
gchatelet added inline comments.
llvm/tools/llvm-exegesis/lib/RegisterAliasing.h
106

Why not implement it this way ? Is this because of knowledge that B has few bits set?

llvm/tools/llvm-exegesis/lib/X86/Target.cpp
181

typo

217

You should provide a rationale for why only {0, 42}

This revision is now accepted and ready to land.Oct 9 2019, 1:08 AM
courbet updated this revision to Diff 223997.Oct 9 2019, 1:22 AM
courbet marked 3 inline comments as done.

Address comments

Thanks !

llvm/tools/llvm-exegesis/lib/RegisterAliasing.h
106

Right, this warrants a comment. Done.

This revision was automatically updated to reflect the committed changes.