Here's the patch for LLVM: http://llvm-reviews.chandlerc.com/D1802 We ended up basing the RNG on the already integrated implementation of MD5, to avoid any external dependencies. We are really just waiting on review of the LLVM patch now that Julian has modified a few things to take care of a performance concern.
- stephen On Wed, Jan 22, 2014 at 3:00 PM, Alp Toker <[email protected]> wrote: > The clang side looks fine, but there's very little context as to what's > going on here so not possible to review it just like that. > > The patch rebases to clang ToT fine but doesn't build due to missing RNG > facilities in LLVM -- could you give a refresher of the status of that with > a link? It's been long enough that not everyone remembers the discussion. > > The last I remember of the discussion was that linking to OpenSSL can be > painful, and it doesn't feel right as a dependency. What are the other > options for pseudo RNG and could we have a simpler scheme? > > That'll help get things moving. > > Alp. > > > > On 22/01/2014 21:48, Julian Lettner wrote: > >> Is there anything stopping this from going forward? >> >> http://llvm-reviews.chandlerc.com/D1803 >> _______________________________________________ >> cfe-commits mailing list >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> > > -- > http://www.nuanti.com > the browser experts > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
