[PATCH] D83088: Introduce CfgTraits abstraction

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 10 18:21:23 PDT 2020


dblaikie added a comment.

This seems like a strange hybrid between a static-polymorphism (with traits) and dynamic polymorphism (with base classes/virtual functions). Could this more readily be just one or the other? (sounds like you're leaning towards dynamic polymorphism)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83088/new/

https://reviews.llvm.org/D83088



More information about the cfe-commits mailing list