[PATCH] D28314: Change sqrt partial inlining to depend on sqrt argument rather than result.

Simon Dardis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 27 11:57:42 PST 2017

Previous message: [PATCH] D28314: Change sqrt partial inlining to depend on sqrt argument rather than result.
Next message: [PATCH] D28314: Change sqrt partial inlining to depend on sqrt argument rather than result.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

sdardis accepted this revision.
sdardis added a comment.
This revision is now accepted and ready to land.

Looking at this again, I'm seeing some possible cases where MIPS can do better but that's no reason to hold this patch back.

LGTM.


https://reviews.llvm.org/D28314

Previous message: [PATCH] D28314: Change sqrt partial inlining to depend on sqrt argument rather than result.
Next message: [PATCH] D28314: Change sqrt partial inlining to depend on sqrt argument rather than result.
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list