Page MenuHomePhabricator

[llvm-exegesis] Ensure that ExecutableFunction are aligned.
ClosedPublic

Authored by courbet on Wed, Oct 9, 6:28 AM.

Details

Summary

Experiments show that this is the alignment we get (for ELF+Linux), but let's ensure that we have it.

Diff Detail

Event Timeline

courbet created this revision.Wed, Oct 9, 6:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptWed, Oct 9, 6:28 AM
Herald added a subscriber: tschuett. · View Herald Transcript
gchatelet added inline comments.Wed, Oct 9, 6:42 AM
llvm/tools/llvm-exegesis/lib/Assembler.cpp
311

assert(isAligned(FunctionAddress, kFunctionAlignment) && "function is not properly aligned");

courbet updated this revision to Diff 224043.Wed, Oct 9, 6:49 AM
courbet marked an inline comment as done.

Address Comments

courbet updated this revision to Diff 224047.Wed, Oct 9, 7:07 AM

make type explicit

gchatelet accepted this revision.Wed, Oct 9, 7:08 AM
This revision is now accepted and ready to land.Wed, Oct 9, 7:08 AM
This revision was automatically updated to reflect the committed changes.