[flang-commits] [flang] [flang] Fix BoxAnalyzer.h for LLVM_ENABLE_EXPENSIVE_CHECKS (PR #96675)

via flang-commits flang-commits at lists.llvm.org
Tue Jun 25 11:28:13 PDT 2024


https://github.com/dpalermo created https://github.com/llvm/llvm-project/pull/96675

Fixes assert seen trying to default number of inlined elements for `SmallVector<T>` but `sizeof(T)` is really big. Seen with -DLLVM_ENABLE_EXPENSIVE_CHECKS.

>From ace151095150c1d8a397386e219d0b64a8977c8b Mon Sep 17 00:00:00 2001
From: Dan Palermo <dan.palermo at amd.com>
Date: Tue, 25 Jun 2024 13:24:24 -0500
Subject: [PATCH] [flang] Fix BoxAnalyzer.h for LLVM_ENABLE_EXPENSIVE_CHECKS

Fixes assert seen trying to default number of inlined elements
for `SmallVector<T>` but `sizeof(T)` is really big.
Seen with -DLLVM_ENABLE_EXPENSIVE_CHECKS.
---
 flang/include/flang/Lower/BoxAnalyzer.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/flang/include/flang/Lower/BoxAnalyzer.h b/flang/include/flang/Lower/BoxAnalyzer.h
index 3b8e2455ff273..8eca7d66a71bf 100644
--- a/flang/include/flang/Lower/BoxAnalyzer.h
+++ b/flang/include/flang/Lower/BoxAnalyzer.h
@@ -97,7 +97,7 @@ struct ScalarDynamicDerived : ScalarSym {
       : ScalarSym{sym}, lens{std::move(lens)} {}
 
 private:
-  llvm::SmallVector<Fortran::lower::SomeExpr> lens;
+  llvm::SmallVector<Fortran::lower::SomeExpr, 1> lens;
 };
 
 struct LBoundsAndShape {



More information about the flang-commits mailing list