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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to