Page MenuHomePhabricator

Update SafeStack TODO in the safestack.cc
ClosedPublic

Authored by ksvladimir on Jun 22 2015, 5:08 AM.

Details

Summary

This patch clarifies the TODO note at the top of safestack.cc and brings it more in sync with what we (the CPI team) actually plan to work on in the future.

Diff Detail

Repository
rL LLVM

Event Timeline

ksvladimir updated this revision to Diff 28104.Jun 22 2015, 5:08 AM
ksvladimir retitled this revision from to Update SafeStack TODO in the safestack.cc.
ksvladimir updated this object.
ksvladimir edited the test plan for this revision. (Show Details)
ksvladimir added reviewers: pcc, jfb.
ksvladimir added a subscriber: Unknown Object (MLST).

Updated to include the cfe-commits list as a subscriber.

pcc accepted this revision.Jun 23 2015, 2:41 PM
pcc edited edge metadata.

LGTM

This revision is now accepted and ready to land.Jun 23 2015, 2:41 PM
This revision was automatically updated to reflect the committed changes.