This is an archive of the discontinued LLVM Phabricator instance.

InstrProf: support PowerPC64
ClosedPublic

Authored by foad on Nov 12 2014, 1:30 PM.

Details

Summary

I don't know anything about profiling but it seems to work out of the
box on PowerPC64. At least "make check-profile" works.

A few tests needed tweaking because PowerPC64 IR declares main with
"define signext i32 @main" instead of just "define i32 @main".

This also fixes the asan asan_and_llvm_coverage_test test, which
compiles with -coverage so requires that a profiling version of
libclang_rt has been built.

Diff Detail

Event Timeline

foad updated this revision to Diff 16111.Nov 12 2014, 1:30 PM
foad retitled this revision from to InstrProf: support PowerPC64.
foad updated this object.
foad edited the test plan for this revision. (Show Details)
foad added reviewers: dexonsmith, kcc.
foad added a subscriber: Unknown Object (MLST).
samsonov accepted this revision.Nov 12 2014, 4:23 PM
samsonov added a reviewer: samsonov.
samsonov added a subscriber: samsonov.

LGTM

cmake/config-ix.cmake
183

Please use 80-cols here.

This revision is now accepted and ready to land.Nov 12 2014, 4:23 PM
foad closed this revision.Nov 13 2014, 1:24 AM