This is an archive of the discontinued LLVM Phabricator instance.

[LLDB] Fixes summary formatter for libc++ map allowing modification of contained value
ClosedPublic

Authored by kpdev42 on Dec 23 2022, 5:10 AM.

Diff Detail

Event Timeline

kpdev42 created this revision.Dec 23 2022, 5:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 23 2022, 5:10 AM
kpdev42 requested review of this revision.Dec 23 2022, 5:10 AM
kpdev42 retitled this revision from [LLDB] Fix for libc++ map allowing modification of contained value to [LLDB] Fixes summary formatter for libc++ map allowing modification of contained value.Dec 23 2022, 6:30 AM
clayborg accepted this revision.Jan 4 2023, 5:14 PM
This revision is now accepted and ready to land.Jan 4 2023, 5:14 PM
omjavaid reopened this revision.Jan 25 2023, 1:28 AM
omjavaid added a subscriber: omjavaid.

This rev has broken lldb-arm-ubuntu buildbot https://lab.llvm.org/buildbot/#/builders/17/builds/33173

This revision is now accepted and ready to land.Jan 25 2023, 1:28 AM

This rev has broken lldb-arm-ubuntu buildbot https://lab.llvm.org/buildbot/#/builders/17/builds/33173

Sorry. I've added a small fix for it https://github.com/llvm/llvm-project/commit/4ae221f5a4d1977f316b7d5f033763f876b471e7
Could you please verify if it works now?