[flang-commits] [flang] [flang][OpenMP] Rename variable `block` to `body`, NFC (PR #156702)
via flang-commits
flang-commits at lists.llvm.org
Wed Sep 3 08:49:21 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-flang-parser
Author: Krzysztof Parzyszek (kparzysz)
<details>
<summary>Changes</summary>
`block` is a global variable that represents the Block parser, so avoid using local variables with the same name.
---
Full diff: https://github.com/llvm/llvm-project/pull/156702.diff
1 Files Affected:
- (modified) flang/lib/Parser/openmp-parsers.cpp (+3-3)
``````````diff
diff --git a/flang/lib/Parser/openmp-parsers.cpp b/flang/lib/Parser/openmp-parsers.cpp
index cc4e59d318bbe..1b7665ce4acf5 100644
--- a/flang/lib/Parser/openmp-parsers.cpp
+++ b/flang/lib/Parser/openmp-parsers.cpp
@@ -1289,9 +1289,9 @@ struct StrictlyStructuredBlockParser {
if (lookAhead(skipStuffBeforeStatement >> "BLOCK"_tok).Parse(state)) {
if (auto epc{Parser<ExecutionPartConstruct>{}.Parse(state)}) {
if (IsFortranBlockConstruct(*epc)) {
- Block block;
- block.emplace_back(std::move(*epc));
- return std::move(block);
+ Block body;
+ body.emplace_back(std::move(*epc));
+ return std::move(body);
}
}
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/156702
More information about the flang-commits
mailing list