[llvm] r177730 - R600: Use legacy (0 * anything = 0) MUL instructions for pow intrinsics
Michel Dänzer
michel.daenzer at amd.com
Tue Mar 26 02:05:37 PDT 2013
On Mon, 2013-03-25 at 10:38 -0700, Rafael Espíndola wrote:
> On 25 March 2013 04:43, Michel Dänzer <michel.daenzer at amd.com> wrote:
> > On Son, 2013-03-24 at 13:48 -0400, Rafael Ávila De Espíndola wrote:
> >> Testcase?
> >
> > There is test/CodeGen/R600/llvm.pow.ll.
>
> What I mean is: Why no testcase was modified by this patch? It is good
> practice to, when possible, to include test changes in the same commit
> that implemented the fix or added the feature.
Yes, I'm trying to get into the habit of that but forgot about it in
this case until after the commit, see r177736.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Debian, X and DRI developer
More information about the llvm-commits
mailing list