[PATCH] D44559: [Sema] Wrong width of result of mul operation

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 26 10:27:17 PDT 2018


lebedev.ri added a comment.

I'm having a very hard time following this thread. In fact, i can't follow the logic at all.

In https://reviews.llvm.org/D44559#1041007, @rjmccall wrote:

> ...
>  This patch, however, is contrary to the design of -Wconversion, which does attempt to avoid warning in cases where the user might reasonably see an operation as "really" being performed in its operand type.  If you want to argue that we should change the design of -Wconversion, that is a broader conversation that you should take to cfe-dev.


Can you at least point out where the current design goals are documented?


https://reviews.llvm.org/D44559





More information about the llvm-commits mailing list