[PATCH] D88201: [DAGCombiner] Add decomposition patterns for Mul-by-Imm.

[Sterling Augustine via Phabricator via llvm-commits]

saugustine added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:16079
+      return false;
+    if (isPowerOf2_64(Imm + 1) || isPowerOf2_64(Imm - 1) ||
+        isPowerOf2_64(1 - Imm) || isPowerOf2_64(-1 - Imm))
----------------
undefined-behaviour sanitizer reports an error at this line when executing the test at llvm-project/llvm/test/CodeGen/PowerPC/mul-const-i64.ll

PPCISelLowering.cpp:16079:27: runtime error: signed integer overflow: 9223372036854775807 + 1 cannot be represented in type 'long'




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88201