martong added a comment. In D69726#2471657 <https://reviews.llvm.org/D69726#2471657>, @vabridgers wrote:
> I was referred to this patch from https://reviews.llvm.org/D86743. I pulled > this patch under review, brought it up to date and pushed to github at > https://github.com/vabridgers/llvm-project-dev.git, branch: vla-fam-fixes. > Everything seems ok on this branch (LITs pass, reproducers from > https://bugs.llvm.org/show_bug.cgi?id=47272 and > https://bugs.llvm.org/show_bug.cgi?id=28450 no longer crash). I can continue > and push a change to Phabricator for review, or @charusso and/or @balazske > could finish this? I didn't want to just push an update without asking first > :/ Cheers! Hi Vince and Artem, I agree that this is an important patch. Unfortunately it got stuck. I see two possible solutions: 1. Vince commandeers this patch. I am not sure about the policy of commandeering a patch and that might be too intrusive. 2. Vince opens a new patch (a copy of this) and gives credit to @Charusso when he commits. I prefer this. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69726/new/ https://reviews.llvm.org/D69726 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
