steakhal added a comment.

Couls you please provide some context about why did we have this suppression in 
the first place?

Other than that, I'm on board with this, but we should really investigate the 
historical reasons why we had this.

If it turns out to be a valid usecase, but a rather special one we could still 
have an analyzer config option making this change optional.

@NoQ do you have any background on this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122841

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

Reply via email to