================
@@ -730,7 +731,9 @@ class YAMLConverter {
}
}
- void convertParams(const ParamsSeq &Params, FunctionInfo &OutInfo) {
+ std::optional<ParamInfo> convertParams(const ParamsSeq &Params,
+ FunctionInfo &OutInfo) {
----------------
compnerd wrote:
While currently we only apply a note to a single implicit parameter (`this` or
`self`), what if we were to extend this in the future? Does it make sense to
take a reference to a `std::optional<ParamInfo>` that we setup as the implicit
instance parameter?
https://github.com/llvm/llvm-project/pull/115021
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
