[PATCH] D116886: [M68k] Instruction selection to choose neg x when mul x -1 (Fix issue 48588)

Douglas Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 10 21:52:06 PST 2022


dougpuob marked an inline comment as done.
dougpuob added inline comments.


================
Comment at: llvm/lib/Target/M68k/M68kISelDAGToDAG.cpp:327
+      if (llvm::isNullConstant(U->getOperand(0)))
+        return false;
+    }
----------------
RKSimon wrote:
> (style) Add a break - even though its the last case in the switch() its good practice to use them.
Agree with you. Thank you for your careful found.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116886



More information about the llvm-commits mailing list