================
@@ -4681,6 +4676,12 @@ void CallArg::copyInto(CodeGenFunction &CGF, Address
Addr) const {
IsUsed = true;
}
+void CodeGenFunction::EmitWritebacks(CodeGenFunction &CGF,
+ const CallArgList &args) {
+ for (const auto &I : args.writebacks())
+ emitWriteback(CGF, I);
+}
----------------
tex3d wrote:
Since this requires a reference to a CodeGenFunction, why not just make this a
regular method instead of a static method?
In fact, after double-checking, it appears that it is a regular method, so why
does it need a separate input argument when it already has `this`?
```suggestion
void CodeGenFunction::EmitWritebacks(const CallArgList &args) {
for (const auto &I : args.writebacks())
emitWriteback(*this, I);
}
```
https://github.com/llvm/llvm-project/pull/109331
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
