[LLVMdev] [PATCH] OpenCL half support

Anton Lokhmotov Anton.Lokhmotov at arm.com
Thu May 17 04:50:38 PDT 2012


Hi David,

Many thanks for the comments!

>       Tha 0xH format should be described in LangRef.html alongside
> 0xK<hex> and 0xM<hex>
Done.

>    Declaration of "int shiftcount" should be moved to smallest nesting
>    possible, right after "if ( const ConstantFP ..." at line 710
> 
>    (The code makes a lot more sense with a good comment on the
> definition
>    of shiftcount.  I prefer to add this comment:
>       int shiftcount; // bit position, in the current word, of the
> next nibble to print.
>    This is a problem with the original code, not the patch.)
Done.

>    The patch removes half case from the code for single and double.
>    So you should remove the "bool isHalf" variable in that spot. (line
> 720)
I'm not sure I get it.  This variable is still needed couple of lines below:

      bool isHalf = &CFP->getValueAPF().getSemantics()==&APFloat::IEEEhalf;
      bool isDouble =
&CFP->getValueAPF().getSemantics()==&APFloat::IEEEdouble;
      bool isInf = CFP->getValueAPF().isInfinity();
      bool isNaN = CFP->getValueAPF().isNaN();
      if (!isHalf && !isInf && !isNaN) {

>     // Either half, or some form of long double.
>     // These appear as a magic letter identifying the type, then a
>     // fixed number of hex digits.
Done.

Does it look better to you?

Many thanks,
Anton.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: half-llvm.patch
Type: application/octet-stream
Size: 6477 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120517/9bd258b0/attachment.obj>


More information about the llvm-dev mailing list