[PATCH] D46179: [X86] Lowering adds/addus/subs/subus intrinsics to native IR (LLVM part)

Tomasz Krupa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 10 23:11:12 PDT 2018


tkrupa added a comment.

During internal review I proposed something like that (~0 - a) < b ? ~0 : a+b.
It gets rid of zext/trunc in addus pattern but introduces additional subtraction which is presumably more costly. Your solution seems much better, I'll change it to that form.


Repository:
  rL LLVM

https://reviews.llvm.org/D46179





More information about the llvm-commits mailing list