[Lldb-commits] [PATCH] D43647: Generate most of the target	properties from a central specification.
    Adrian Prantl via Phabricator via lldb-commits 
    lldb-commits at lists.llvm.org
       
    Thu Feb 22 17:12:56 PST 2018
    
    
  
aprantl added a comment.
I like Pavel's template syntax. It's not clear to me how to best implement the registering of the properties this way though. I think we want to avoid static intializers. Given how TargetProperties::TargetProperties() is implemented, I could imagine we could just make the properties members of the class, we just need some way to generate a for-each-property loop to handle the initialization. I'll think about this some more.
Thanks for your feedback!
Repository:
  rL LLVM
https://reviews.llvm.org/D43647
    
    
More information about the lldb-commits
mailing list