https://github.com/5chmidti requested changes to this pull request.

Looks good, but the `operand` node may also be in an unevaluated context, you 
only guard against the non `++/--` expr being in an unevaluated context here. 
Fix: add `unless(IsInUnevaluatedContext)` to both `expr().bind("operand")`.
Example: `if (foo < static_cast<decltype(foo++)>(bar)) {}`

https://github.com/llvm/llvm-project/pull/85849
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to