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

[Roland Scheidegger via Phabricator via llvm-commits]

sroland added a comment.

In https://reviews.llvm.org/D46179#1093151, @tkrupa wrote:

> We're currently discussing the possible solutions for the JIT pipeline issue with @DavidKreitzer.


Thanks for taking a look. I mean, it was quite easy to adapt our code when the intrinsics removed had more or less obvious, not complicated fallbacks (which we most likely already had in our code anyway), such as the sse2 pmin/pmax or pabs functions, but it gets really cumbersome (with quite some potential for issues due to the requirement to match what autoupgrade would do exactly) with the ones which are rather complex to emulate.


Repository:
  rL LLVM

https://reviews.llvm.org/D46179