Szelethus added a comment.

In D140387#4007751 <https://reviews.llvm.org/D140387#4007751>, @balazske wrote:

> This patch and D140395 <https://reviews.llvm.org/D140395> is (almost) the 
> same code as D135360 <https://reviews.llvm.org/D135360> and D135247 
> <https://reviews.llvm.org/D135247>. The changes are separated for the 
> different checkers. Tests are added at the second patch.

Well, that is confusing. Which patches should I review? Can you please only 
have a single opened differential per change, and a single patch stack? If you 
decided that this patch is **the ONE** for the `ErrnoModeling` changes, please 
abandon the other patch with the very same change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140387

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

Reply via email to