<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Nov 8, 2017 at 3:42 PM, Chad Versace <span dir="ltr"><<a href="mailto:chadversary@chromium.org" target="_blank">chadversary@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed 08 Nov 2017, Jason Ekstrand wrote:<br>
> On Wed, Nov 8, 2017 at 1:40 PM, Chad Versace <[1]<a href="mailto:chadversary@chromium.org">chadversary@chromium.org</a>><br>
<div><div class="h5">> wrote:<br>
><br>
> On Tue 07 Nov 2017, Dylan Baker wrote:<br>
> > Quoting Eric Engestrom (2017-11-07 07:25:53)<br>
> > > On Wednesday, 2017-11-01 13:49:03 -0700, Chad Versace wrote:<br>
> > > > I tested this in a setup where the builddir was outside of the<br>
> srcdir.<br>
> > > > ---<br>
> > > > src/intel/vulkan/meson.build | 12 ++++++++++++<br>
> > > > 1 file changed, 12 insertions(+)<br>
> > > ><br>
> > > > diff --git a/src/intel/vulkan/meson.build b/src/intel/vulkan/<br>
> meson.build<br>
> > > > index ff24e304ef5..e8b7f407507 100644<br>
> > > > --- a/src/intel/vulkan/meson.build<br>
> > > > +++ b/src/intel/vulkan/meson.build<br>
> > > > @@ -48,6 +48,18 @@ intel_icd = custom_target(<br>
> > > > install : true,<br>
> > > > )<br>
> > > ><br>
> > > > +dev_icd = custom_target(<br>
> > > > + 'dev_icd',<br>
> > > > + input : 'anv_icd.py',<br>
> > > > + output : 'dev_icd.@0@.json'.format(<wbr>target_machine.cpu()),<br>
> > ><br>
> > > Strictly speaking, shouldn't that be `host_machine` [1] ?<br>
> > > I don't see how one would do a canadian build of mesa though, so<br>
> > > host == target should always be true.<br>
> ><br>
> > That's my fault. There are (or were) a number of cases where I used<br>
> target<br>
> > instead of host, that can also be a follow up.<br>
> ><br>
> > In any case:<br>
</div></div>> > Acked-by: Dylan Baker <[2]<a href="mailto:dylan@pnwbakers.com">dylan@pnwbakers.com</a>><br>
<span class="">><br>
> I build Mesa (with autotools) where host == x86_64 but target == armv7a.<br>
><br>
><br>
> You're using dev_icd with a cross-compile? Yikes! I mean, it can work, but<br>
> that's not what I would have expected. Also, why are you building anv and<br>
> targetting armv7a???? Does that even work?<br>
<br>
</span>Two many interacting topics!<br>
<br>
Eric said: "I don't see how one would do a canadian build of mesa<br>
though, so host == target should always be true". Mesa, not anvil.<br>
I replied that I cross-compile Mesa for ARM.<br></blockquote><div><br></div><div>Ok, that makes more sense.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On host-vs-target: My host machine and target machine are rarely the<br>
same. But, for Anvil, of course, the two machines do always have the<br>
same architecture.<br>
<br>
Regardless, the icd filename should always contain the target<br>
architecture, because run the icd *on the target*.<br>
</blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Fair enough. Pedantry in these areas is perfectly reasonable.<br></div></div>