[PATCH] D93447: [Flang][openmp][openacc] Extend CheckNoBranching to handle branching provided by LabelEnforce.

[sameeran joshi via Phabricator via llvm-commits]

sameeranjoshi marked an inline comment as done.
sameeranjoshi added inline comments.


================
Comment at: flang/lib/Semantics/check-directive-structure.h:45
     currentStatementSourcePosition_ = statement.source;
+    if (statement.label.has_value()) {
+      labels_.insert(*statement.label);
----------------
clementval wrote:
> Braces should be removed for simple if
Was it a comment on fir-dev?
I think flang[1] allows and fir-dev follows llvm coding standard[2].

[1] https://github.com/llvm/llvm-project/blob/master/flang/docs/C%2B%2Bstyle.md#layout
[2] https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop-statements


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93447