krytarowski added a comment.

In https://reviews.llvm.org/D23881#525420, @mgorny wrote:

> In https://reviews.llvm.org/D23881#525411, @krytarowski wrote:
>
> > In https://reviews.llvm.org/D23881#525405, @mgorny wrote:
> >
> > > In https://reviews.llvm.org/D23881#525401, @krytarowski wrote:
> > >
> > > > The proper way to fix it is to install CheckAtomic from LLVM/Clang.
> > > >
> > > > See https://reviews.llvm.org/D22999
> > >
> > >
> > > Wasn't that rejected?
> >
> >
> > There was rejected a proposal to add CheckAtomic copy to LLDB. Not a 
> > proposal to install CheckAtomic for more customers of LLVM.
>
>
> Ah, sorry, I didn't read that thoroughly. Should I create a patch for that, 
> or will you handle it from here?


Please create it and I will test it on NetBSD.

Please add reviewers from https://reviews.llvm.org/D22999


Repository:
  rL LLVM

https://reviews.llvm.org/D23881



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

Reply via email to