baloghadamsoftware added a comment.

The unit test you added is very welcome. However, please still extend the 
current tests (which are for the signed) also for the unsigned as well. Since 
this is a core change in the analyzer, let us wait for @NoQ as well before 
accepting it.


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

https://reviews.llvm.org/D77802



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

Reply via email to