jcai19 added a comment.

In D63623#1556161 <https://reviews.llvm.org/D63623#1556161>, @lebedev.ri wrote:

> In D63623#1552716 <https://reviews.llvm.org/D63623#1552716>, @jcai19 wrote:
>
> > In D63623#1552679 <https://reviews.llvm.org/D63623#1552679>, @lebedev.ri 
> > wrote:
> >
> > > Why is this in android module?
> > >  Is that android-specific behavior, or posix?
> >
> >
> > I implemented it for Andriod as requested, but it would be completely fine 
> > to move it if it is better to place the check at a different location. 
> > Where do you suggest we should move this check to? Thanks.
>
>
> Either `misc` or `bugprone` or a new `posix` module.


Sounds good. Thanks for the clarification.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63623



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

Reply via email to