Displaying 4 results from an estimated 4 matches for "ishalf".
2012 May 17
0
[LLVMdev] [PATCH] OpenCL half support
...s 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::IEEEdo...
2012 May 11
2
[LLVMdev] [PATCH] OpenCL half support
...hiftcount. 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.)
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)
Line 762. Update the comment
// Some form of long double. These appear as a magic letter identifying
Before:
// Some form of long double. These appear as a magic letter identifying
// the type, then a fixed number of hex digits.
A...
2012 May 17
3
[LLVMdev] [PATCH] OpenCL half support
...tcount; // 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->getValueA...
2012 May 17
0
[LLVMdev] [PATCH] OpenCL half support
...he 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;
> >...