AlexeySachkov added inline comments.
================ Comment at: clang/include/clang/AST/Type.h:1827-1830 + if (Dependent) + Deps |= TypeDependence::Dependent | TypeDependence::Instantiation; + if (InstantiationDependent) + Deps |= TypeDependence::Instantiation; ---------------- sammccall wrote: > AlexeySachkov wrote: > > @ilya-biryukov, Is this code snippet correct? > > > > It seems to be, that it should look like: > > ``` > > if (Dependent) > > Deps |= TypeDependence::Dependent; > > if (InstantiationDependent) > > Deps |= TypeDependence::Dependent | TypeDependence::Instantiation; > > ``` > I agree that seems clearer, but ISTM they are equivalent because a dependent > type is always instantiation-dependent (right?) > > Are you seeing related problems? > Are you seeing related problems? I though I was seeing a related problem, but it turned out that I wasn't. Looking at the code again with a clear head, I believe that everything is correct here Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71920/new/ https://reviews.llvm.org/D71920 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits