[clang] [llvm] [LLVM][IR] Add native vector support to ConstantInt & ConstantFP. (PR #74502)

Nikita Popov via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 5 10:35:03 PST 2023


================
@@ -343,7 +343,7 @@ static bool verifyTripCount(Value *RHS, Loop *L,
     // If the RHS of the compare is equal to the backedge taken count we need
     // to add one to get the trip count.
     if (SCEVRHS == BackedgeTCExt || SCEVRHS == BackedgeTakenCount) {
-      ConstantInt *One = ConstantInt::get(ConstantRHS->getType(), 1);
+      ConstantInt *One = ConstantInt::get(ConstantRHS->getIntegerType(), 1);
----------------
nikic wrote:

I don't really follow here. Even if you rename the overload on ConstantInt, there will still be the method inherited from `Value::getType()`, and using that method here should work (and be forward-compatible with vector ConstantInt), because `ConstantInt::get` doesn't actually require that the argument is an IntegerType.

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


More information about the cfe-commits mailing list