[PATCH] D88723: [flang] Fix build errors for clang-10

Andrzej Warzynski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 2 01:56:52 PDT 2020


awarzynski created this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
awarzynski requested review of this revision.

Two NFCs that fix building with Clang-10. These are only wornings, but
since Flang sets `-Werror`, we need to get rid of this.  As per
flang/README.md, Clang-10 is one of the officially supported compilers.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D88723

Files:
  flang/lib/Lower/OpenACC.cpp
  flang/runtime/io-stmt.h


Index: flang/runtime/io-stmt.h
===================================================================
--- flang/runtime/io-stmt.h
+++ flang/runtime/io-stmt.h
@@ -162,7 +162,7 @@
   bool lastWasUndelimitedCharacter{false};
 };
 template <>
-class ListDirectedStatementState<Direction::Input>
+struct ListDirectedStatementState<Direction::Input>
     : public FormattedIoStatementState {
 public:
   // Skips value separators, handles repetition and null values.
Index: flang/lib/Lower/OpenACC.cpp
===================================================================
--- flang/lib/Lower/OpenACC.cpp
+++ flang/lib/Lower/OpenACC.cpp
@@ -99,7 +99,6 @@
   if (loopDirective.v == llvm::acc::ACCD_loop) {
     auto &firOpBuilder = converter.getFirOpBuilder();
     auto currentLocation = converter.getCurrentLocation();
-    llvm::ArrayRef<mlir::Type> argTy;
 
     // Add attribute extracted from clauses.
     const auto &accClauseList =


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88723.295760.patch
Type: text/x-patch
Size: 928 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201002/ab397618/attachment.bin>


More information about the llvm-commits mailing list