[PATCH] D150955: [clang][AST] Propagate the value-dependent bit for VAArgExpr.

Fri May 19 11:34:42 PDT 2023

hokein added a comment.

Thanks for the review.

In D150955#4356580 <https://reviews.llvm.org/D150955#4356580>, @aaron.ballman wrote:

> LGTM assuming CI is okay with it -- the patch did not apply so precommit CI never ran. I'm surprised no tests break as a result of this because it changes both C and C++ behavior. The changes should have a release note, too.

This patch was based on my another fix (this probably explains why it failed to apply on pre-merge testing CI). I have test it with `ninja check-clang`.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150955