================
@@ -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