[PATCH] D139274: Store OptTable::Info::Name as a StringRef

serge via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 6 05:48:11 PST 2022


serge-sans-paille added a comment.

In D139274#3974176 <https://reviews.llvm.org/D139274#3974176>, @thakis wrote:

> Reverted in de4b6a1bc64db33643f001ad45fae7b92b4a4688 <https://reviews.llvm.org/rGde4b6a1bc64db33643f001ad45fae7b92b4a4688> for now. (Note that there was a follow-up fix in e50a60d7349de151bd2b06d85a79201ebc372d8a <https://reviews.llvm.org/rGe50a60d7349de151bd2b06d85a79201ebc372d8a> that I reverted in 13e061e73e1b067589d3bd680fc29297d7e4ec8d <https://reviews.llvm.org/rG13e061e73e1b067589d3bd680fc29297d7e4ec8d>. When you reland, you'll likely want to cherry-pick that into your reland.)

You've been quicker than I am. Thanks for the revert.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139274/new/

https://reviews.llvm.org/D139274



More information about the llvm-commits mailing list