[PATCH] Add support to promote f16 to f32

Ahmed Bougacha ahmed.bougacha at gmail.com
Wed Apr 1 10:35:03 PDT 2015


Awesome, I wanted to do this next, thanks for working on this =)

I'll look into this closer, but a couple of very high level not-really-questions first:

- what about other types?  I just removed the generic f32->f64 promotion a few days ago (in anticipation for such a PromoteFloat legalizer).  I'm not sure we can test that in-tree, so there's probably not much point in worrying about this.
- might I ask: what's your use case?  In particular, do you intend to change clang in any way?

In the meantime, you might be interested in http://reviews.llvm.org/D8648 for tests, as I tried to cover as many IR/generic ISD opcodes as possible.  The patch itself just focuses on ops promotion on AArch64 (where half is legal).

-Ahmed


http://reviews.llvm.org/D8755

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






More information about the llvm-commits mailing list