[Lldb-commits] [PATCH] D140262: [lldb][TypeSystemClang][NFC] Introduce TemplateParameterInfos::ArgumentMetadata
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Dec 19 01:17:56 PST 2022
labath accepted this revision.
labath added inline comments.
This revision is now accepted and ready to land.
================
Comment at: lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.h:334
+ struct ArgumentMetadata {
+ char const *const name = nullptr;
+ };
----------------
This may depend on where you're going with this, but right now it seems like this `const` here is overkill, as one can express the notion of "const metadata" by making the whole ArgumentMetadata class const.
That obviously won't work if your actual goal is to have some kind of "mutable" metadata, but then I'd like to hear more about what is that going to be used for.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140262/new/
https://reviews.llvm.org/D140262
More information about the lldb-commits
mailing list