Nativepolytope: When intersecting, check easy cases where one of the polytopes are universal first. This also prevents that an internal Eigen assertion is raised in cases where an empty and a non-empty matrix are concatenated (somehow only relevant for gcc 9.3 in debug mode...).

src/storm/storage/geometry/NativePolytope.cpp

@@ -190,6 +190,11 @@ namespace storm {
             template <typename ValueType>
             std::shared_ptr<Polytope<ValueType>> NativePolytope<ValueType>::intersection(std::shared_ptr<Polytope<ValueType>> const& rhs) const{
                 STORM_LOG_THROW(rhs->isNativePolytope(), storm::exceptions::InvalidArgumentException, "Invoked operation between a NativePolytope and a different polytope implementation. This is not supported");
+                if (this->isUniversal()) {
+                    return rhs;
+                } else if (rhs->isUniversal()) {
+                    return std::make_shared<NativePolytope<ValueType>>(*this);
+                } else {
                     NativePolytope<ValueType> const& nativeRhs = dynamic_cast<NativePolytope<ValueType> const&>(*rhs);
                     EigenMatrix resultA(A.rows() + nativeRhs.A.rows(), A.cols());
                     resultA << A,
@@ -199,6 +204,7 @@ namespace storm {
                                nativeRhs.b;
                     return std::make_shared<NativePolytope<ValueType>>(EmptyStatus::Unknown, std::move(resultA), std::move(resultb));
                 }
+            }
             
             template <typename ValueType>
             std::shared_ptr<Polytope<ValueType>> NativePolytope<ValueType>::intersection(Halfspace<ValueType> const& halfspace) const{