[PATCH] D66887: [test-suite][WIP] Add GCC C Torture Suite as External Test Suite

Tue Oct 8 08:26:06 PDT 2019

lenary marked 4 inline comments as done.
lenary added inline comments.

================
Comment at: SingleSource/Regression/C/gcc-c-torture/README:14-17
+# Licensing
+
+The testcases in SingleSource/Regression/C/gcc-c-torture/execute are covered by
+the GPL. See the files whose names start with COPYING for copying permission.
----------------
kristof.beyls wrote:
> Given licensing information is also present in SingleSource/Regression/C/gcc-c-torture/execute/LICENSE.TXT and called out in the top-level LICENSE.TXT file in the test-suite, I don't think there's much value in repeating the information here too. Maybe best to delete this part?
Sure, deleted.

================
Comment at: SingleSource/Regression/C/gcc-c-torture/execute/LICENCE.TXT:1-2
+The testcases in this directory are covered by the GPL. See the files whose
+names start with COPYING for copying permission.
----------------
kristof.beyls wrote:
> it seems the file name here is still LICENCE.TXT rather than LICENSE.TXT, which would be preferred for consistency with the rest of the test-suite?
I'm really sorry about you having to note this twice. I must have been having a british/american spelling moment and missed the C/S difference. Updated.

Repository:
  rT test-suite

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

https://reviews.llvm.org/D66887