[PATCH] D108623: [flang] Fix non-deterministic line output function

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 4 16:48:13 PDT 2021


Meinersbur added inline comments.


================
Comment at: flang/lib/Semantics/check-declarations.cpp:1156
+  auto checkDefinedOperatorArgs = 
+      [&](SourceName opName, const Symbol &specific, const Procedure &proc) {
+    bool arg0Defined {CheckDefinedOperatorArg(opName, specific, proc, 0)};
----------------
ijan1 wrote:
> Meinersbur wrote:
> > Nothing is implicitly captured.
> I thought I had left a comment but it seems like it was unsubmitted. The `this` pointer is being captured here.
OK. Could have used `[this]`, it is always captured by value even with `[&]`. No need to change after it having been committed.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108623/new/

https://reviews.llvm.org/D108623



More information about the llvm-commits mailing list