yaxunl added a comment. In D67509#1677586 <https://reviews.llvm.org/D67509#1677586>, @yaxunl wrote:
> In D67509#1677528 <https://reviews.llvm.org/D67509#1677528>, @tra wrote: > > > Looks like CUDA test-suite is triggering the assertion added by this patch: > > > > http://lab.llvm.org:8011/builders/clang-cuda-build/builds/37301/steps/ninja%20build%20simple%20CUDA%20tests/logs/stdio > > > I am taking a look. I can reproduce similar asserts locally. It seems the assertion I added `assert(!(HasD || HasH) || (NeedsD == HasD && NeedsH == HasH));` is not always true. Since we do not have this assert before, I removed it. I will study what causes it to assert and post it later. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67509/new/ https://reviews.llvm.org/D67509 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
