This is an archive of the discontinued LLVM Phabricator instance.

Remove unnecessary demangling operation (hopefully NFC)
ClosedPublic

Authored by aprantl on Feb 27 2019, 8:23 AM.

Details

Summary

This extra call to the demangler doesn't affect the performance of C++ because the result is being cached anyway; but I'm working on a patch to the Swift branch that uses extra contextual information to provide a more accurate demangling result. In that case this call would be extra and unnecessary work.

Diff Detail

Repository
rL LLVM

Event Timeline

aprantl created this revision.Feb 27 2019, 8:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 27 2019, 8:23 AM
clayborg accepted this revision.Feb 27 2019, 10:15 AM
This revision is now accepted and ready to land.Feb 27 2019, 10:15 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptFeb 27 2019, 2:54 PM