[PATCH] D59254: [RFC] Implementation of Clang randstruct

Connor Kuehl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 1 21:46:44 PDT 2019


connorkuehl added a comment.

In D59254#1565961 <https://reviews.llvm.org/D59254#1565961>, @jfb wrote:

> Do you intend on moving this forward? It seems like a Good Thing overall, and I'd love to help review some more.


Yes. I'm sorry it's been in limbo so long. I haven't touched base with the rest of my team so I can't speak for them, but I'm interested in moving this along. I've been busy settling into work and post-university life lately.

>From your initial review it sounds like there's quite a bit to be done by way of providing sufficient test coverage for this and some code cleanup. A few other kind contributors have left us advice on our github repo for some blocking issues with designated initializers.

I'll be reviewing all the advice we've been given here and on our repo and figuring out the best way to start making progress on this again. There's definitely a good basis here.

That said, I'm still interested in and would appreciate any advice or pointers to resources. I don't suppose there's an LLVM/Clang mentorship type program?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59254





More information about the cfe-commits mailing list