Hi Evan,

> > It already had ECX or'd in.
> 
> Then plz write something like "0xb8 | 1" and comment it so it's more  
> clear (to me at least :-).

this is from the original version - the version I committed before
the holidays already did 0xb8 | 1, and the version I just committed
does it even better :)

> > Does the attached patch seem OK?
> 
> Yes, thx.

Thanks for the review!  I just committed it.

Ciao,

Duncan.
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to