<p dir="ltr"><br>
On Mar 6, 2016 2:26 PM, "Marek Olšák" <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>> wrote:<br>
><br>
> Hi Oded,<br>
><br>
> I prefer "if" over #ifdef. The idea is that everybody should be able<br>
> to test if the compilation succeeds without a BE machine. #ifdef<br>
> disallows that.<br>
><br>
> R600_BIG_ENDIAN can be moved to r600_pipe_common.h.<br>
><br>
> Marek<br>
><br>
OK, no problem. <br>
I'll move the define as part of the next patch set I'll send. <br>
Oded </p>
<p dir="ltr">><br>
> On Sun, Mar 6, 2016 at 9:01 AM, Oded Gabbay <<a href="mailto:oded.gabbay@gmail.com">oded.gabbay@gmail.com</a>> wrote:<br>
> > Hi,<br>
> ><br>
> > Do you mind if I totally remove R600_BIG_ENDIAN global variable and<br>
> > instead use in all places #ifdef PIPE_ARCH_BIG_ENDIAN ?<br>
> ><br>
> > It's just that:<br>
> ><br>
> > 1. Checking for R600_BIG_ENDIAN is an extra check which can be<br>
> > eliminated using #ifdef<br>
> ><br>
> > 2. Some files, e.g r600_texture.c, don't know R600_BIG_ENDIAN so I<br>
> > need to use the #ifdef anyway.<br>
> ><br>
> > 3. Other drivers in mesa use #ifdef<br>
> ><br>
> > Thanks,<br>
> ><br>
> >      Oded<br>
</p>