[PATCH] D108711: [flang] Take result length into account in ApplyElementwise folding

Peter Klausler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 25 10:14:16 PDT 2021


klausler accepted this revision.
klausler added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!



================
Comment at: flang/lib/Evaluate/fold-implementation.h:1048
     -> std::optional<Expr<RESULT>> {
+  auto resultLength = ComputeResultLength(operation);
   auto &leftExpr{operation.left()};
----------------
braces, please


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108711



More information about the llvm-commits mailing list