This is an archive of the discontinued LLVM Phabricator instance.

[NFC][Sanitizer] Rename BufferedStackTrace::FastUnwindStack
ClosedPublic

Authored by yln on Feb 22 2019, 11:42 AM.

Details

Summary

FastUnwindStack -> UnwindFast
SlowUnwindStack -> UnwindSlow
Stack is redundant, verb should come first.

SlowUnwindStackWithContext(uptr pc, void *context, u32 max_depth) ->
SlowUnwindStack
WithContext is redundant, since it is a required parameter.

Overview revision:
https://reviews.llvm.org/D58536

Event Timeline

yln created this revision.Feb 22 2019, 11:42 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 22 2019, 11:42 AM
Herald added subscribers: llvm-commits, Restricted Project, jrtc27 and 3 others. · View Herald Transcript
vitalybuka accepted this revision.Feb 22 2019, 1:50 PM
This revision is now accepted and ready to land.Feb 22 2019, 1:50 PM
This revision was automatically updated to reflect the committed changes.