This is an archive of the discontinued LLVM Phabricator instance.

Update SanitizerSpecialCaseList to use renamed functions in base class.
ClosedPublic

Authored by hctim on Oct 31 2017, 5:39 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

hctim edited reviewers, added: vlad.tsyrklevich; removed: kcc.Oct 31 2017, 6:02 PM

Why is there a cyclical dependency? D39485 doesn't delete SpecialCaseList::inSection()?

hctim added a comment.Nov 1 2017, 10:52 AM

Why is there a cyclical dependency? D39485 doesn't delete SpecialCaseList::inSection()?

This line calles the protected member function, not the public one :) Protected member function got the /s/inSection/inSectionBlame treatment.

hctim updated this revision to Diff 121754.Nov 6 2017, 10:49 AM

Merge for submission.

hctim updated this revision to Diff 121755.Nov 6 2017, 10:51 AM

Whoops - update wrong commit. Reverting these changes.

vlad.tsyrklevich accepted this revision.Nov 7 2017, 12:56 PM
This revision is now accepted and ready to land.Nov 7 2017, 12:56 PM