[PATCH] D100347: Fix the const overload of IgnoreExprNodes

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 13 09:05:45 PDT 2021


gribozavr2 added a comment.

> Trying arc land I realize I do not...

Pushed as 595c3758e47042c876e7390d09642ea050ba6dbf <https://reviews.llvm.org/rG595c3758e47042c876e7390d09642ea050ba6dbf>.

> it's also unclear to me where the tests for this are so I couldn't add a new test, is that OK?

Yes, unfortunately we don't have unit tests for the AST.

> More importantly, the bug exists in LLVM 12, what is the policy of patching the back branches in this project?

Please take a look at https://llvm.org/docs/HowToReleaseLLVM.html#merge-requests .


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100347



More information about the cfe-commits mailing list