nickdesaulniers added a comment.

I'm super confused between the commit message and initial hunk, that seem to 
make sense and probably should go in clang-11 if it's not too late, and the 
additional tests for modules which the commit message does not address.  Were 
these meant to be separate commits, because otherwise it looks like you 
uploaded unrelated stuff.  Are C++20 modules broken with `asm goto`, or are you 
just adding additional test coverage for that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88195

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

Reply via email to