shafik added a comment.

In D130058#3701485 <https://reviews.llvm.org/D130058#3701485>, @smeenai wrote:

> In D130058#3698213 <https://reviews.llvm.org/D130058#3698213>, @shafik wrote:
>
>>> Given that we have a non-obvious (at least to me) issue in a widely used 
>>> third-party library, would we consider giving users some way to opt out of 
>>> this error, at least as a transition tool?
>>
>> Thank your feedback, this is very helpful. I won't have time to dig into 
>> this in detail till tomorrow but it does not look like there is a nice fix 
>> here. I think we may want to consider turning this into a warning for some 
>> transition period.
>
> Following up because we have an internal integration blocked on this. Are we 
> planning to turn this into a warning soon, or should I just put in local 
> workarounds in the meantime?

I plan on having a PR up tomorrow and hopefully turn around will be quick.


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

https://reviews.llvm.org/D130058

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

Reply via email to