[PATCH] D87188: [InstCombine] Canonicalize SPF to abs intrinc

Sanjay Patel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 22 13:32:13 PDT 2020


spatel added a comment.

In D87188#2348326 <https://reviews.llvm.org/D87188#2348326>, @nikic wrote:

> Reopening this so we don't forget...
>
> I believe @spatel is working on the cost modelling. I did not have much luck tracking down the miscompile, at least did not spot anything incriminating in the llvm-diff.

Yes, I'm working through the mess of the TTI cost model with things like:
c963bde0152a <https://reviews.llvm.org/rGc963bde0152a741d58b9e1e9ac485d8f2e6c6d58>
It's a slog...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87188



More information about the cfe-commits mailing list