On Wed, Oct 19, 2022 at 10:14:28AM -0700, Andrew Pinski wrote: > Do the testcases really need to be changed rather than adding new testcases? > Usually it is better if the testcases not change unless really needed > to be. That is do these testcases pass without being changed? If not > this seems not backwards compatible change and is not something which > we should do. Otherwise you should just add new testcases instead.
Yes, that is another reason why adding parameters to random builtins is not a good idea :-) s/random/only vaguely related/, if you want. This also makes all existing code using these builtins invalid. If you need such testcase changes, that is a red flag. Segher