================
@@ -39,12 +39,17 @@ struct AccessPath {
   //   temporary object materialized via this MaterializeTemporaryExpr.
   const llvm::PointerUnion<const clang::ValueDecl *,
                            const clang::MaterializeTemporaryExpr *>
-      P;
+      P = nullptr;
 
 public:
+  AccessPath() {};
----------------
Xazax-hun wrote:

Why was this default constructor introduced? I would not want to make this type 
nullable unless we have a strong reason. We could consider having an 
`std::optional<AccessPath>` if the need for nullability is local. 

https://github.com/llvm/llvm-project/pull/179093
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to