martong accepted this revision.
martong added a comment.

In D86691#2241604 <https://reviews.llvm.org/D86691#2241604>, @nullptr.cpp wrote:

> In D86691#2241401 <https://reviews.llvm.org/D86691#2241401>, @Szelethus wrote:
>
>> Nice, thank you! Did you stumble across this, or found it with a tool?
>
> I just happened to see these code:-).
> Maybe we should write a checker to check this situation? I will start doing 
> this!

Good idea! And thanks for the fix! :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86691

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

Reply via email to