[flang-commits] [flang] [flang][Parser] Convert applyMem to invoke non-void members (PR #119782)

Krzysztof Parzyszek via flang-commits flang-commits at lists.llvm.org
Mon Dec 16 06:03:42 PST 2024


================
@@ -141,7 +141,7 @@ collect the values that they return.
 * `applyLambda([](&&x){}, p1, p2, ...)` is the same thing, but for lambdas
   and other function objects.
 * `applyMem(mf, p1, p2, ...)` is the same thing, but invokes a member
-  function of the result of the first parser for updates in place.
+  function of the result of the first parser.
----------------
kparzysz wrote:

It returns the value returned by the member call, so for member functions returning void (as it was before) it wouldn't work.  For member functions returning reference to `*this` or something equivalent it should be fine.

Is modifying in-place is still needed?  If so I'd need to separate the code from this PR into its own parser.

https://github.com/llvm/llvm-project/pull/119782


More information about the flang-commits mailing list