[PATCH] D34303: [InstCombine] Make FPMathOperator working with ConstantExpression

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 16:52:34 PDT 2017


You're welcome. That was the main reason I pointed it out. The real bug
number was of course easy to find from the test file name.

~Craig

On Fri, Jun 16, 2017 at 4:46 PM, Davide Italiano via Phabricator <
reviews at reviews.llvm.org> wrote:

> davide added a comment.
>
> In https://reviews.llvm.org/D34303#782959, @davide wrote:
>
> > In https://reviews.llvm.org/D34303#782953, @craig.topper wrote:
> >
> > > Is that the wrong bug link in the description?
> >
> >
> > Oh, yes, sigh :( https://bugs.llvm.org//show_bug.cgi?id=33453
>
>
> Thanks for catching it, I amended my commit message :)
>
>
> https://reviews.llvm.org/D34303
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170616/09edb022/attachment.html>


More information about the llvm-commits mailing list