This is an archive of the discontinued LLVM Phabricator instance.

#if 0 for DWARFDebugInfo::Find() as it is unused
ClosedPublic

Authored by jankratochvil on Nov 18 2017, 2:19 PM.

Details

Summary

I would have to update DWARFDebugInfo::Find() for the DWZ support I am going to submit. But these two methods are currently not used anywhere so there is no way to test them and updating them is then a bit pointless.

Or should the code be removed instead of #if 0?

Diff Detail

Repository
rL LLVM

Event Timeline

jankratochvil created this revision.Nov 18 2017, 2:19 PM
clayborg requested changes to this revision.Nov 19 2017, 10:23 AM

Feel free to remove any unused code. No need for review on dead code removal. So just remove the code, don't add #if 0

This revision now requires changes to proceed.Nov 19 2017, 10:23 AM
This revision was automatically updated to reflect the committed changes.