This is an archive of the discontinued LLVM Phabricator instance.

[JSONNodeDumper] Do not print mangled names for local variables (PR49111)
ClosedPublic

Authored by nikic on Dec 22 2021, 8:06 AM.

Details

Summary

Mangled names are not meaningful for variables with local storage, and may not be well defined (getting the mangled name for VLA crashes the mangler).

The recommended solution from https://github.com/llvm/llvm-project/issues/49111 is to drop the mangledName for this case.

Diff Detail

Event Timeline

nikic requested review of this revision.Dec 22 2021, 8:06 AM
nikic created this revision.
This revision is now accepted and ready to land.Dec 22 2021, 12:05 PM
This revision was landed with ongoing or failed builds.Dec 22 2021, 11:55 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptDec 22 2021, 11:55 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript