[PATCH] D68230: [UpdateCCTestChecks] Allow asm in output with --allow-asm

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 18 05:52:40 PDT 2020


greened added a comment.

Getting back to this.  From the (very long!) thread on end-to-end testing, people are generally supportive of the idea.  The main question is where tests should be located.  That question is othogonal to this patch.  We currently use the functionality of this patch downstream and I'm guessing others would also find it useful.  Since the idea of end-to-end testing seems to be supported in the community, can we merge this patch and then have a follow-up discussion about where such tests should live upstream?  Getting this merged now would allow others to experiment with/use the functionality as discussion about where to place end-to-end tests continues.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68230





More information about the llvm-commits mailing list