[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