jcsxky added a comment.

In D159412#4636813 <https://reviews.llvm.org/D159412#4636813>, @donat.nagy 
wrote:

> Please add a testcase that demonstrates this issue (fails when your change in 
> MemRegion.cpp isn't added) and shows that your commit fixes it.

Thanks for your advice.  We use this api in our own project and miss some cases 
which leads the incorrect result. I will abandon this patch later.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159412

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to