[PATCH] D106294: [flang] Implement the runtime portion of the UNPACK intrinsic

Mark LeAir via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 19 12:12:22 PDT 2021


mleair added inline comments.


================
Comment at: flang/runtime/transformational.cpp:524
     mask.IncrementSubscripts(maskAt);
-    field.IncrementSubscripts(fieldAt);
+    if (field.rank() != 0) { // scalar "field" argument
+      field.IncrementSubscripts(fieldAt);
----------------
Same comment as before. llvm style does not like braces for single code blocks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106294



More information about the llvm-commits mailing list