[PATCH] D153106: [CSKY] Optimize multiplication with immediates

Ben Shi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 19 02:35:58 PDT 2023


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


================
Comment at: llvm/lib/Target/CSKY/CSKYISelLowering.cpp:1386
+  // Omit if data size exceeds.
+  const unsigned DataSize = Subtarget.hasE2() ? 32 : 16;
+  if (VT.getSizeInBits() > DataSize)
----------------
zixuan-wu wrote:
> As this line code shows, it try to avoid data type larger than machine register size.  And, CSKY is always 32-bit machine.
Thanks. I have fixed this issue in my new uploaded patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153106



More information about the llvm-commits mailing list