In http://reviews.llvm.org/D8367#141841, @olista01 wrote:

> You have marked this as depending on http://reviews.llvm.org/D4602, but this 
> patch includes all of the changes from http://reviews.llvm.org/D4602, is this 
> intentional?


Yeah, I'm not sure it makes sense to do this in two parts, so let's say it 
supersedes http://reviews.llvm.org/D4602.  If you want I can isolate the f64 
changes back.

> This patch changes the behaviour for half<->int conversions, but I only see a 
> test the for half->int direction. Could you add some more?


Added, thanks!

-Ahmed


http://reviews.llvm.org/D8367

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to