[PATCH] D106292: [flang] Implement the runtime portion of the CSHIFT intrinsic

[Mark LeAir via Phabricator via llvm-commits]

mleair added inline comments.


================
Comment at: flang/runtime/transformational.cpp:190
+    SubscriptValue sourceAt{lb + (j + shiftCount) % extent};
+    if (sourceAt < 0) {
+      sourceAt += extent;
----------------
llvm style recommends no braces for single statement blocks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106292