This is an archive of the discontinued LLVM Phabricator instance.

Use llvm:Twine instead of std::to_string.
ClosedPublic

Authored by chaoren on Apr 27 2016, 3:50 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

chaoren updated this revision to Diff 55345.Apr 27 2016, 3:50 PM
chaoren retitled this revision from to Use llvm:Twine instead of std::to_string..
chaoren updated this object.
chaoren added reviewers: lhames, ovyalov, chandlerc.
chaoren added a subscriber: llvm-commits.

Good to have a reason in the commit message/patch description - I assume
it's because std::to_string isn't available on some platforms? At least
that's what I recall seeing in previous patches.

chaoren updated this revision to Diff 55348.Apr 27 2016, 4:08 PM

Add reason.

ovyalov accepted this revision.Apr 27 2016, 5:02 PM
ovyalov edited edge metadata.

LGTM

This revision is now accepted and ready to land.Apr 27 2016, 5:02 PM
This revision was automatically updated to reflect the committed changes.