[PATCH] D106893: [flang] Disallow BOZ literal constants as output list items

Peter Klausler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 27 10:46:07 PDT 2021


klausler accepted this revision.
klausler added inline comments.
This revision is now accepted and ready to land.


================
Comment at: flang/lib/Semantics/expression.cpp:159
   int GetRank(std::size_t) const;
-  bool IsBOZLiteral(std::size_t i) const {
-    return std::holds_alternative<BOZLiteralConstant>(GetExpr(i).u);
-  }
+  bool IsBOZLiteral(std::size_t i) const { return IsBozLiteral(GetExpr(i)); }
   void SayNoMatch(const std::string &, bool isAssignment = false);
----------------
It would be great if you could use the same capitalization for both of these predicates.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106893



More information about the llvm-commits mailing list