[Lldb-commits] [PATCH] D40133: elf-core: Convert remaining register context to use register set maps
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Nov 17 02:31:27 PST 2017
labath added a comment.
In https://reviews.llvm.org/D40133#927655, @krytarowski wrote:
> I'm just thinking whether change code like `CoreRegset::PPC_VMX` to `CoreRegset::PPC::VMX`. I'm planning to add `CoreRegset::X86_64::XSAVE` in future.
That's an interesting idea, but I don't see how would that be possible -- enum values can't have nested names. I suppose we could fake that by making `CoreRegset` and `CoreRegset::X86_64` namespaces, but then the name of the enum type would have to be something different (`CoreRegset::Type`?). I am not sure if that's worth the trouble.
I don't think we will have that many register sets, so it's probably fine to have flat namespaces for arch/regset combinations.
More information about the lldb-commits