[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