Fznamznon added a comment. In D114080#3158368 <https://reviews.llvm.org/D114080#3158368>, @Fznamznon wrote:
> In D114080#3158323 <https://reviews.llvm.org/D114080#3158323>, @erichkeane > wrote: > >> I note that this is missing a test, otherwise I don't see any issues with it >> from my end. > > There is a test SemaSYCL/zero-length-arrays.cpp. For some reason after I > updated this revision Phabricator doesn't show the full diff, but it shows > both commits. The first commit with main changes and the test is available > here - https://reviews.llvm.org/D114080?id=387904 . It seems I was able to fix that. Now the link to this revision shows the full diff. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114080/new/ https://reviews.llvm.org/D114080 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
