[PATCH] D143624: Inlining: Run the legacy AlwaysInliner before the regular inliner.

[Amara Emerson via Phabricator via cfe-commits]

aemerson added a comment.

In D143624#4312905 <https://reviews.llvm.org/D143624#4312905>, @dmgreen wrote:

> Hello. It sounds like it is really close to being OK. The combine of the shift just seem to make things more difficult.
>
> The `icmp ult i1 %cmp4.i, true` is just a not, would it help if it was actually an xor? Or if the `not(icmp sgt)` was changed to a slt earlier?
>
> I was taking a look at the example but I am not super sure what to suggest. Would it be best if the code that detect min/max looked through not's?

I posted an attempt at this: https://reviews.llvm.org/D149725


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143624