[PATCH] D55843: [CodeGen] Handle mixed-width ops in mixed-sign mul-with-overflow lowering

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 18 11:45:42 PST 2018


efriedma added a comment.

This looks reasonable.



================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:1265
+    NeedSExt = Op1Info.Signed;
+    NeedZExt = !Op1Info.Signed;
+  } else if (Op1Info.Width > Op2Info.Width) {
----------------
This is a weird way to express this... could you instead compute the width of the signed operand and the unsigned operand, and compare those?


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

https://reviews.llvm.org/D55843





More information about the cfe-commits mailing list