<div dir="ltr">+Samuel Pitoiset <div><br></div><div><br></div><div class="gmail_extra"><div class="gmail_quote">2018-06-11 22:31 GMT+02:00 Mauro Rossi <span dir="ltr"><<a href="mailto:issor.oruam@gmail.com" target="_blank">issor.oruam@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Bas,<div><br></div><div>commit [1] removed a check 

<span style="font-size:small;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">on 'supported' attribute</span>

 in src/intel/vulkan/anv_<wbr>entrypoints_gen.py</div><div><br></div><div>Should the check on 'supported' attribute be removed also in src/amd/vulkan/radv_<wbr>entrypoints_gen.py ?</div><div><br></div><div>Thanks for your feedback</div><div>Mauro</div><div><br></div><div>[1] <a href="https://cgit.freedesktop.org/mesa/mesa/commit/?id=63525ba730e3d8a466d7f6382a2b91f4c75dd171" target="_blank">https://cgit.freedesktop.<wbr>org/mesa/mesa/commit/?id=<wbr>63525ba730e3d8a466d7f6382a2b91<wbr>f4c75dd171</a></div><div><br></div><div><div><br></div></div></div>
</blockquote></div><br></div><div class="gmail_extra"><div class="gmail_extra">Hi Sam, Bas,</div><div class="gmail_extra"><br></div><div class="gmail_extra">there is an important matter regarding Android builds (android-x86)</div><div class="gmail_extra"><br></div><div class="gmail_extra">I have a series of patches for Android makefiles to build radv for that platform,</div><div class="gmail_extra">they are building but they require the change in src/amd/vulkan/radv_entrypoints_gen.py </div><div class="gmail_extra">to have the necessary entrypoints and vulkan apps starting to work.</div><div class="gmail_extra"><br></div><div class="gmail_extra">What I forgot to say is that I have the Android building rules ready to submit to mesa-dev,</div><div class="gmail_extra">but they require to build libLLVM with a different name e.g libLLVM60 or libLLVM_mesa and set the correct HAVE_LLVM properly</div><div class="gmail_extra"><br></div><div class="gmail_extra">The patches themselves would break the Android build for Intel, because amd tree is built unconditionally,</div><div class="gmail_extra">but the libLLVM"for mesa" shared library is not in place in AOSP, Intel builds and not even in android-x86 oreo,</div><div class="gmail_extra"><br></div><div class="gmail_extra">so I wanted to start discussing about how to integrate ravd for Android in a way that does not break other drivers builds,</div><div class="gmail_extra">if there are other interested users.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Mauro</div></div></div>