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

[Craig Topper via Phabricator via cfe-commits]

craig.topper added a comment.

Fair point, what is the default signedness of char?

FWIW, all these  warn in gcc. So they seem to be just checking purely based on the int promotion without any concern for the original size?

  unsigned short foo(unsigned char a) {
    return a * a;
  }
  
  signed short bar(signed char a) {
    return a * a;
  }
  
  signed short foo(signed char a) {
    return a + a;
  }
  
  unsigned short foo(unsigned char a) {
    return a + a;
  }


Repository:
  rC Clang

https://reviews.llvm.org/D44559