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

Andrew V. Tischenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 26 04:11:36 PDT 2018


avt77 added a comment.

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

> No, I still oppose this patch.


OK, we have 2 possibilities here (fmpov):

1. Forget about the issue and don't do anything now - it is not a bug
2. Return the width based on real analyze of mul args:
  - signed vs. unsigned
  - chars, shorts, etc.

What do you suggest to do? Or maybe there are other variants?


https://reviews.llvm.org/D44559





More information about the llvm-commits mailing list