On Monday, February 12, 2018 5:26:15 PM PDT Dongwon Kim wrote:
> Add a new option, '--pciid' to override a pci id of the target arch
> to support cross-architecture shader compilation. Not like "-p" option,
> it is for accepting any GFX devices supported by the driver.
> 
> Setting both "-p" and "--pciid" is blocked to avoid conflict.
> 
> Signed-off-by: Dongwon Kim <dongwon....@intel.com>
> ---
>  run.c | 44 ++++++++++++++++++++++++++++++++++++++++++--
>  1 file changed, 42 insertions(+), 2 deletions(-)

Hi Dongwon,

It looks like this does the exact same thing as -p, but it accepts
arbitrary numerical PCI IDs instead of a platform name.  IMHO, I think
we should just make -p try parsing it as a number of it doesn't match
any of the names.  Then either -p bdw or -p 0x1616 would work.

It seems much less complicated that way.

--Ken

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to