================
@@ -2646,6 +2647,8 @@ class FunctionDecl : public DeclaratorDecl,
   bool isTypeAwareOperatorNewOrDelete() const;
   void setIsTypeAwareOperatorNewOrDelete(bool IsTypeAwareOperator = true);
 
+  UsualDeleteParams getUsualDeleteParams() const;
----------------
ojhunt wrote:

@kees is it reasonable to have this here but the struct definition be in a 
separate header? Moving the definition to Decl.h, requires a bunch of other 
related definitions to be moved - I think it would be reasonable to do so if 
we're going to make this present directly in Decl?

@andykaylor I think this should be an optional return - e.g. 
`std::optional<UsualDeleteParams>` or we could smooth out the declaration w/o 
the definition with `bool getUsualDeleteParams(UsualDeleteParams *) const`. I'd 
prefer `optional<UsualDeleteParams>` given the alternation of `true` and 
`false` meaning success or fail in different places.

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

Reply via email to