This is an archive of the discontinued LLVM Phabricator instance.

[Demangle][Rust] Parse path backreferences
ClosedPublic

Authored by tmiasko on Jun 1 2021, 6:57 AM.

Diff Detail

Event Timeline

tmiasko created this revision.Jun 1 2021, 6:57 AM
tmiasko requested review of this revision.Jun 1 2021, 6:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 1 2021, 6:57 AM
dblaikie accepted this revision.Jun 3 2021, 8:24 PM

Looks good - probably 3 separate commits, though. (handy if they're separate reviews too in the future - just a bit easier to check tests, etc)

llvm/lib/Demangle/RustDemangle.cpp
509–511

Maybe split this in two - add the two different places for backrefs separately, with separate tests? Oh, three different places (also demangleConst)?

This revision is now accepted and ready to land.Jun 3 2021, 8:24 PM
tmiasko updated this revision to Diff 350425.Jun 7 2021, 3:00 PM
tmiasko retitled this revision from [Demangle][Rust] Parse backreferences to [Demangle][Rust] Parse path backreferences.

Parse path backreferences

tmiasko marked an inline comment as done.Jun 7 2021, 3:05 PM
tmiasko added inline comments.
llvm/lib/Demangle/RustDemangle.cpp
509–511

I split patch into D103847 and D103848, while adding extra tests that exercise recursion limit in each production.

This revision was landed with ongoing or failed builds.Jun 8 2021, 1:03 AM
This revision was automatically updated to reflect the committed changes.
tmiasko marked an inline comment as done.