================ @@ -54,7 +54,8 @@ class MockArgList { } template <class T> LIBC_INLINE T next_var() { - ++arg_counter; + arg_counter = ---------------- michaelrj-google wrote:
I don't think anything directly depends on its output, but also the comments on it specifically say that it's for testing. Given how small `ArgList` is I'd rather have another copy than run into problems because a patch in the future didn't expect the testing version to be used in real code. https://github.com/llvm/llvm-project/pull/96369 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits