================
@@ -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());
----------------
AaronBallman wrote:

I think we should make the math operate on a `T` consistently, especially given 
that `Offset` and `Index` are already a `T`. The mixture of types is a bit of a 
code smell IMO.

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