aaron.ballman added a comment.

In D88833#3222331 <https://reviews.llvm.org/D88833#3222331>, @estan wrote:

> @lebedev.ri @aaron.ballman I think the opt-in sounds good too. Are we waiting 
> for @fiesh to implement this, or is the patch abandoned? Would be sad if it 
> was stranded the way https://reviews.llvm.org/D31130 was, since I think many 
> people are skipping this check due to diagnostics that are for all intents 
> and purposes "out of their control".

My expectation was that @fiesh would be updating the review if they wanted this 
to land. If they indicate they're no longer interested in working on the patch, 
then I think it's fine for you to commandeer the patch. But you should give 
them a chance to speak up in case they're still intending to finish this. 
@fiesh, are you expecting to work on this further?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88833

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to