commit bfc6a2dc7ae3fe2587f6008254de1e2e26f0cd71
author Peter Klausler <pklausler@nvidia.com> Tue Nov 23 13:37:15 2021 -0800
committer Copybara-Service <copybara-worker@google.com> Fri Nov 26 12:36:54 2021 -0800
tree a6979ff90c3a9a2aae6d022733b33a389d7b9ab6
parent fd7b264d9b11990383141fddac03d4075a6c8507

[flang] Fix reversed comparison in RESHAPE() runtime

RESHAPE() fails inappropriately at runtime if the source array
is larger than the result -- which is perfectly valid -- because
of an obviously reversed comparison of their numbers of elements
is activating the runtime asserts meant for the opposite case
(source smaller than result).

Differential Revision: https://reviews.llvm.org/D114474

GitOrigin-RevId: 45a8caf1cdf6a9cbe6dd183df62f22c57e3310ae

runtime/transformational.cpp

diff --git a/runtime/transformational.cpp b/runtime/transformational.cpp
index 42381ee..0ac1d46 100644
--- a/runtime/transformational.cpp
+++ b/runtime/transformational.cpp
@@ -385,7 +385,7 @@
   std::size_t elementBytes{source.ElementBytes()};
   std::size_t sourceElements{source.Elements()};
   std::size_t padElements{pad ? pad->Elements() : 0};
-  if (resultElements < sourceElements) {
+  if (resultElements > sourceElements) {
     RUNTIME_CHECK(terminator, padElements > 0);
     RUNTIME_CHECK(terminator, pad->ElementBytes() == elementBytes);
   }