[PATCH] D49273: [InstCombine] Expand the simplification of pow() into exp2()

Evandro Menezes via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 30 11:04:44 PDT 2018


Given what AMDGPULibCalls::evaluateScalarMathFunc()does, I double 
checked all the transcendentals in it and the only one missing from the 
transforms of pow()is exp2().  However, I don't have access to a Windows 
machine, so either I commit it and hope for the best or wait for a 
charitable soul (Reid?) to confirm this fix.

-- 
Evandro Menezes

On 08/30/2018 12:55 PM, Sanjay Patel via Phabricator wrote:
> spatel accepted this revision.
> spatel added a comment.
> This revision is now accepted and ready to land.
>
> LGTM
>
> Did someone confirm that producing exp2() was the cause of the bot failure?
>
>
> https://reviews.llvm.org/D49273
>
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180830/e5bfce8f/attachment.html>


More information about the llvm-commits mailing list