We already have this exact code in lldb/cmake/LLDBConfig.txt. This fix looks fine, but I see no reason to have it twice, so can you delete the LLDB version?
On Mon, Apr 17, 2017 at 7:01 AM Kamil Rytarowski via Phabricator < revi...@reviews.llvm.org> wrote: > krytarowski added a comment. > > Is this just GNU specific? BSD moved to 64-bit off_t on 32-bit platforms > 20+ years ago. > > It's perhaps no-op, but it might be noted in the commit message what > platforms are supposed to be improved. > > > Repository: > rL LLVM > > https://reviews.llvm.org/D32125 > > > >
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits