[flang-commits] [PATCH] D147224: [flang] move ASSOCIATED intrinsic optional TARGET handling

[Slava Zakharin via Phabricator via flang-commits]

vzakhari accepted this revision.
vzakhari added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: flang/lib/Lower/CustomIntrinsicCall.cpp:281
+  // TARGET is absent.
+  fir::ExtendedValue targetExv = getOperand(1, /*loadOperand*/ false);
+  mlir::Value targetBase = fir::getBase(targetExv);
----------------
nit: usually, it would look like `/*loadOperand=*/false` (with `=` and without a space before `false`).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147224