[llvm] [LLVM][ADT] Explicitly convert size_t values to SmallVector's size type (PR #77939)

Duncan P. N. Exon Smith via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 12 10:31:39 PST 2024


================
@@ -99,8 +99,9 @@ template <class Size_T> class SmallVectorBase {
   ///
   /// This does not construct or destroy any elements in the vector.
   void set_size(size_t N) {
-    assert(N <= capacity());
-    Size = N;
+    auto n = static_cast<Size_T>(N);
+    assert(n <= capacity());
----------------
dexonsmith wrote:

IMO, @browneee 's suggestion here is more clear / easier to follow. `n` vs. `N` is pretty subtle.

https://github.com/llvm/llvm-project/pull/77939


More information about the llvm-commits mailing list