<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 3, 2017 at 3:18 PM, Lionel Landwerlin <span dir="ltr"><<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 03/10/17 21:21, Chad Versace wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue 03 Oct 2017, Lionel Landwerlin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 03/10/17 19:13, Jason Ekstrand wrote:<br>
<br>
     On Tue, Oct 3, 2017 at 9:43 AM, Chad Versace <[1]<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</a>><br>
     wrote:<br>
<br>
         On Tue 03 Oct 2017, Lionel Landwerlin wrote:<br>
         > v2: Simplify offset enum computation (Jason)<br>
         ><br>
         > Signed-off-by: Lionel Landwerlin <[2]<a href="mailto:lionel.g.landwerlin@intel.com" target="_blank">lionel.g.landwerlin@intel.<wbr>com</a>><br>
         > ---<br>
         >  src/vulkan/util/vk_util.h | 6 ++++++<br>
         >  1 file changed, 6 insertions(+)<br>
         ><br>
         > diff --git a/src/vulkan/util/vk_util.h b/src/vulkan/util/vk_util.h<br>
         > index 2ed601f881e..8c8cb64d513 100644<br>
         > --- a/src/vulkan/util/vk_util.h<br>
         > +++ b/src/vulkan/util/vk_util.h<br>
         > @@ -199,4 +199,10 @@ __vk_find_struct(void *start, VkStructureType<br>
         sType)<br>
         ><br>
         >  uint32_t vk_get_driver_version(void);<br>
         ><br>
         > +#define VK_EXT_OFFSET (1000000000UL)<br>
         > +#define vk_enum_extension(__enum) \<br>
         > +   ((__enum) >= VK_EXT_OFFSET ? ((((__enum) - VK_EXT_OFFSET) /<br>
         1000UL) + 1) : 0)<br>
         > +#define vk_enum_offset(__enum) \<br>
         > +   ((__enum) >= VK_EXT_OFFSET ? ((__enum) % 1000) : (__enum))<br>
<br>
         The macro functions, when called, look like regular functions due to<br>
         being lowercase. But they don't behave like functions; their arguments<br>
         suffer from the multiple evaluation disease.<br>
<br>
         Please rename the macros to be all uppercase, so callers' expectations<br>
         will be set correctly. Or, even better, define them as inline<br>
         functions.<br>
<br>
<br>
     +1 to static inline<br>
<br>
Done locally.<br>
</blockquote>
Cool. Waiting to see it appear in wip/djeath/ycbcr_conversion.<br>
</blockquote>
<br></div></div>
Ah...<br>
I didn't actually test that (with all the other commits on top).<br>
<br>
Unfortunately that's breaking a bit the way we index formats :<br>
<br>
<a href="https://github.com/djdeath/mesa/blob/wip/djdeath/ycbcr_conversion/src/intel/vulkan/anv_formats.c#L49" rel="noreferrer" target="_blank">https://github.com/djdeath/mes<wbr>a/blob/wip/djdeath/ycbcr_conve<wbr>rsion/src/intel/vulkan/anv_<wbr>formats.c#L49</a></blockquote><div><br></div><div>Right... That's a bummer.  Macros it is, I guess. <br></div></div></div></div>