[PATCH] D147267: [NFC][DebugInfo] Make Descriptions constexpr

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 18 14:56:42 PDT 2023


dblaikie added a comment.

> Assuming I've actually measured that there are no significant differences in instructions retired or max-rss (and that the remaining code changes needed to support a dynamic number of operands don't change things significantly) it seems like the more maintainable approach of using a SmallVector for the operands and not forcing things into constexpr boxes would be preferable.

Yeah, seems likely a good way to go. Thanks for looking into it!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147267



More information about the llvm-commits mailing list