From f0a62c90bcb59bbea04cbb3cb4cc818cda9251ce Mon Sep 17 00:00:00 2001
From: Gael Guennebaud <g.gael@free.fr>
Date: Mon, 29 Sep 2014 10:27:51 +0200
Subject: [PATCH] Avoid comparisons between different index types.

---
 Eigen/src/Core/PermutationMatrix.h | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/Eigen/src/Core/PermutationMatrix.h b/Eigen/src/Core/PermutationMatrix.h
index d5113ce19..4846f2ae1 100644
--- a/Eigen/src/Core/PermutationMatrix.h
+++ b/Eigen/src/Core/PermutationMatrix.h
@@ -303,7 +303,9 @@ class PermutationMatrix : public PermutationBase<PermutationMatrix<SizeAtCompile
     /** Constructs an uninitialized permutation matrix of given size.
       */
     explicit inline PermutationMatrix(Index size) : m_indices(size)
-    {}
+    {
+      eigen_internal_assert(size <= NumTraits<StorageIndexType>::highest());
+    }
 
     /** Copy constructor. */
     template<typename OtherDerived>
@@ -375,7 +377,8 @@ class PermutationMatrix : public PermutationBase<PermutationMatrix<SizeAtCompile
       : m_indices(other.nestedPermutation().size())
     {
       eigen_internal_assert(m_indices.size() <= NumTraits<StorageIndexType>::highest());
-      for (StorageIndexType i=0; i<m_indices.size();++i)
+      StorageIndexType end = StorageIndexType(m_indices.size());
+      for (StorageIndexType i=0; i<end;++i)
         m_indices.coeffRef(other.nestedPermutation().indices().coeff(i)) = i;
     }
     template<typename Lhs,typename Rhs>