ychen added a comment.
Thanks for the review.
================
Comment at: clang/test/Sema/exprs.c:19
if (0) {
- 0 / (0 ? 1 : 0); // expected-warning {{expression result unused}}
}
----------------
ychen wrote:
> aaron.ballman wrote:
> > Why did we lose this diagnostic (and the above comment about not changing
> > the test)?
> Because the behavior change here is that only reachable code is diagnosed. I
> should have kept the comment.
I've put it back.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103938/new/
https://reviews.llvm.org/D103938
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
