On Wed, Dec 22, 2021 at 10:02:02PM +0100, Bernhard Reutner-Fischer wrote:
> You should state how you tested the patch. Please refer to
> https://gcc.gnu.org/contribute.html#testing

I though about this, but when gcc syncs changes with upstream [1], it
does not syncs tests, even though they were there [2].

  [1]: https://github.com/gcc-mirror/gcc/commit/b667dd7017a8
  [2]: https://github.com/llvm/llvm-project/commit/0c81a62d9d76

That's why I though that test can be ommitted in this case (I've added a
test for llvm in [3]).

  [3]: https://reviews.llvm.org/D116176

So what is the right way here?
- migrate all tests
- write test only for setbuffer()
- do not add any tests, since they are covered in llvm repo

-- 
Azat.

Reply via email to