[PATCH] D106721: [AArch64] Implemnt MSVC __mulh and __umulh builtins and corresponding IR level intrinsics

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 24 11:26:09 PDT 2021


efriedma added a comment.

We won't call compiler-rt for i128 multiply on aarch64.  It's not worthwhile under any circumstances.

Pattern-matching 64/64->128 multiply in SelectionDAG legalization has been reliable in practice, as far as I know.  And even it misses due to weird behavior in instcombine or something like that, we only end up with one or two extra mla instructions, so not a big deal.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106721



More information about the llvm-commits mailing list