================
@@ -1437,7 +1437,7 @@ bool OffsetHelper(InterpState &S, CodePtr OpPC, const T 
&Offset,
     return false;
   };
 
-  unsigned MaxOffset = MaxIndex - Ptr.getIndex();
+  T MaxOffset = T::from(MaxIndex - Ptr.getIndex(), Offset.bitWidth());
----------------
erichkeane wrote:

Should MaxIndex just be a `T` as well?  I wonder if this function should just 
be 'fully dedicated' to being `T` instead of just this 1 variable here?

https://github.com/llvm/llvm-project/pull/68679
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to