This is an archive of the discontinued LLVM Phabricator instance.

Remove instance reference to static member function
ClosedPublic

Authored by zturner on Sep 29 2015, 1:56 PM.

Details

Reviewers
clayborg
Summary

Hi Greg,

This patch is pretty straightforward, but I'm posting it here because the fact that you were explicitly casting to reference through an instance suggests that maybe there's a design problem you overlooked. OTOH, it's possible this patch is fine. LMK

Diff Detail

Event Timeline

zturner updated this revision to Diff 36033.Sep 29 2015, 1:56 PM
zturner retitled this revision from to Remove instance reference to static member function.
zturner updated this object.
zturner added a reviewer: clayborg.
zturner added a subscriber: lldb-commits.
clayborg accepted this revision.Oct 5 2015, 12:40 PM
clayborg edited edge metadata.

Back from vacation, sorry for the delay. Looks good.

This revision is now accepted and ready to land.Oct 5 2015, 12:40 PM
zturner closed this revision.Oct 15 2015, 1:52 PM