rapidsna wrote: > Pretty sizable and just did a quick scan... but I am not sure how I feel > about using dependence, an already difficult to grok/comprehend bit that is > designed exclusively for some pretty rough situations in templates, with > this. I'm leaning towards 'we should find a different way'.
@erichkeane I'm open to alternative suggestions here, but I'm not sure how else we can cheaply identify which types actually need a TreeTransform pass to resolve the placeholder. TypeDependence::LateParsedAttr propagates that information through wrapper types, and ProcessLateParsedTypeAttributes skips fields whose type carries no placeholder. Without it, we'd have to recursively walk every declaration's full type tree just to decide whether a transform is needed at all. https://github.com/llvm/llvm-project/pull/212906 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
