[PATCH] D60662: [ConstantRange] Simplify unittests after getSetSize was removed

Sun Apr 14 01:29:14 PDT 2019

MaskRay added a comment.

> The commit message was
> 
>> The last use of this helper method was removed by rL302385 <https://reviews.llvm.org/rL302385>.
> 
> Which is clearly not true, as this commit removed more uses of that method, that weren't testing the method itself, but using it. Therefore i'm questioning the whole removal. Moving it into a static function in tests is another, unrelated matter.

That commit didn't remove more uses of that method. The last library use was removed two years ago and no use has been added since then. I deleted it, and then realized there was a unittest relying on it.

> and the way the test was rewritten to avoid it looks more complicated than it was with the function, at least to me..
> 
> Please just revert those two commits.

I agree that the last commit made it more complex than it should. I apologize for that. So I created this revision. However, "more complicated than it was with the function" is your opinion and I'm not convinced of that judgement. So, I'd like to get more input from others, especially those who removed the last few library uses.

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60662/new/

https://reviews.llvm.org/D60662