Page MenuHomePhabricator

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

Differential D21983

Add C++ dependencies to xray runtime
ClosedPublic
Actions

Authored by dberris on Jul 5 2016, 12:35 AM.

Details

Reviewers
aaron.ballman
echristo
rnk
Commits
rG39baab932677: Add C++ dependencies to xray runtime
rC275368: Add C++ dependencies to xray runtime
rL275368: Add C++ dependencies to xray runtime
Summary

Depends on D21982 which implements the in-memory logging implementation of the
XRay runtime. These additional changes also depends on D20352 which adds the
bulk of XRay flags/dependencies when using the -fxray-instrument flag from
Clang.

Diff Detail

Repository
rL LLVM

Event Timeline

dberris updated this revision to Diff 62721.Jul 5 2016, 12:35 AM
dberris retitled this revision from to Add C++ dependencies to xray runtime.
dberris updated this object.
dberris added reviewers: echristo, rnk, aaron.ballman.
dberris added parent revisions: D20352: Add XRay flags to Clang, D21982: [compiler-rt][XRay] Initial per-thread inmemory logging implementation.
dberris added a subscriber: cfe-commits.
Herald added a subscriber: mehdi_amini. · View Herald TranscriptJul 5 2016, 12:35 AM
dberris added a child revision: D21987: [XRay] Implement `llvm-xray extract`, start of the llvm-xray tool.Jul 5 2016, 3:56 AM
dberris updated this revision to Diff 63040.Jul 7 2016, 1:54 AM
  • Check first whether D is actually not nullptr
dberris updated this revision to Diff 63042.Jul 7 2016, 1:59 AM

Rebase

aaron.ballman accepted this revision.Jul 7 2016, 5:33 AM
aaron.ballman edited edge metadata.

LGTM (once other deps land).

This revision is now accepted and ready to land.Jul 7 2016, 5:33 AM
dberris updated this revision to Diff 63916.Jul 13 2016, 9:58 PM
dberris edited edge metadata.

Rebase

dberris updated this revision to Diff 63917.Jul 13 2016, 9:59 PM
  • Undo unnecessary formatting change
Closed by commit rL275368: Add C++ dependencies to xray runtime (authored by dberris). · Explain WhyJul 13 2016, 10:06 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
cfe/
trunk/
lib/
Driver/
19 lines

Diff 63919

View Options

cfe/trunk/lib/Driver/Tools.cpp

Loading...