baloghadamsoftware added a comment. In https://reviews.llvm.org/D22374#575579, @NoQ wrote:
> Ping!~ Did my idea sound completely wrong to you? :) > > Does https://reviews.llvm.org/D25660 depend on this patch? And/or did you > find another workaround? > > upd.: I also thought that this deserves more code comments - the purpose of > the new mechanism should be more obvious by reading the code. Sorry, since it turned out that https://reviews.llvm.org/D25660 does not depend on this patch I downprioritized it. By some mistake I missed the notification about your patch-over-patch. It seems OK to me. https://reviews.llvm.org/D22374 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
