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

[Pete Steinfeld via Phabricator via llvm-commits]

PeteSteinfeld added inline comments.


================
Comment at: flang/runtime/transformational.cpp:190
+    SubscriptValue sourceAt{lb + (j + shiftCount) % extent};
+    if (sourceAt < 0) {
+      sourceAt += extent;
----------------
mleair wrote:
> llvm style recommends no braces for single statement blocks.
I didn't think that we were following this recommendation for flang code.  This file, in particular is full of single statement blocks that are enclosed by braces.  There's one at line 162, another at line 167, line 232, ...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106292