lukel97 wrote: > You can also remove the constructor that takes only the canonical IV? VPCostContext's constructor still uses it, but I was able to change that to take in a VPlan too, should be done now in a9e599bc51b6 https://github.com/llvm/llvm-project/pull/152969