This is an archive of the discontinued LLVM Phabricator instance.

AST: Update reference temporary mangling
ClosedPublic

Authored by majnemer on May 6 2014, 2:10 PM.

Details

Summary

Update our mangling to match the discussion on cxx-abi-dev.

This involves using a seq-id instead of an optional number.

Diff Detail

Event Timeline

majnemer updated this revision to Diff 9133.May 6 2014, 2:10 PM
majnemer retitled this revision from to AST: Update reference temporary mangling.
majnemer updated this object.
majnemer added a reviewer: rsmith.
majnemer added a subscriber: Unknown Object (MLST).
rsmith accepted this revision.May 6 2014, 2:41 PM
rsmith edited edge metadata.
rsmith added inline comments.
lib/AST/ItaniumMangle.cpp
3433

Since you're moving this code anyway... why the static_cast? Also, C not c. Also, using a char here is a little weird. :)

This revision is now accepted and ready to land.May 6 2014, 2:41 PM
majnemer closed this revision.May 19 2014, 7:23 AM
majnemer updated this revision to Diff 9553.

Closed by commit rL208140 (authored by @majnemer).