asmith added a comment.

> @rjmccall wrote:
>  I think you're missing what I'm asking.  If LLVM accepts this feature, it 
> will become our collective responsibility as a project to keep it working.  
> You have a large external correctness test suite for this feature.  It does 
> not sound like you intend to open-source that test suite; instead, you intend 
> to extract a small number of unit tests from it and add those to the LLVM 
> test suite.  So I'm asking if you're at least going to have an external CI 
> bot which will run the full test suite for this feature to ensure it hasn't 
> been broken by the last day of commits.  It does not seem reasonable to 
> expect that the few unit tests you extract will themselves be sufficient to 
> keep the feature tested.

Microsoft's SEH tests are open source and there was a suggestion in one of the 
threads on llvm-dev to run them on the buildbots. We can look into that as a 
follow on to this patch.

https://lists.llvm.org/pipermail/llvm-dev/2019-November/136695.html
https://lists.llvm.org/pipermail/llvm-dev/2020-April/140614.html


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80344

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to