umesh.kalappa0 added a comment.

In D142872#4090317 <https://reviews.llvm.org/D142872#4090317>, @nikic wrote:

> You also shouldn't need to introduce any new handling for the fwrapv option. 
> I'm not sure how exactly it is currently threaded through, but there must be 
> existing handling for it already. For example, there is 
> isSignedOverflowDefined() on LangOptions.

Make sense here .


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

https://reviews.llvm.org/D142872

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

Reply via email to