This is an archive of the discontinued LLVM Phabricator instance.

[memprof] Only insert dynamic shadow load when needed
ClosedPublic

Authored by tejohnson on May 2 2022, 12:41 PM.

Details

Summary

We don't need to insert a load of the dynamic shadow address unless there
are interesting memory accesses to profile.

Split out of D124703.

Diff Detail

Event Timeline

tejohnson created this revision.May 2 2022, 12:41 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 2 2022, 12:41 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
tejohnson requested review of this revision.May 2 2022, 12:41 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 2 2022, 12:41 PM
tejohnson edited the summary of this revision. (Show Details)May 2 2022, 12:43 PM
snehasish accepted this revision.May 2 2022, 12:47 PM

lgtm

llvm/test/Instrumentation/HeapProfiler/no-instrumentation.ll
15

Can we also add a check for the log line "MEMPROF done instrumenting:"? Otherwise this test won't fail if memprof instrumentation doesn't run.

This revision is now accepted and ready to land.May 2 2022, 12:47 PM
tejohnson marked an inline comment as done.May 2 2022, 12:53 PM
tejohnson updated this revision to Diff 426495.May 2 2022, 12:53 PM

Address comment

This revision was landed with ongoing or failed builds.May 2 2022, 1:36 PM
This revision was automatically updated to reflect the committed changes.