3 small updates and fixes#368
Conversation
qwertychouskie
changed the title
This line in features.txt currently has `supports`, not `support`. (GL_ARB_texture_compression_bptc also uses `supports`, but has other issues that break its parsing right now anyways.) I plan on also fixing these inconsistencies in upstream features.txt, but for the sake of the "date added" display, it's still beneficial to fix here as well.
Last remnants of `nv30` tracking were removed upstream with https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/42506
- Move Vivante with other hardware drivers, instead of being after translation and software drivers - For vendors with multiple drivers, use consistent "older hardware -> newer hardware" ordering (AMD and Nvidia were already ordered this way, but not Broadcom and Intel) - Show translation category before software category
|
@MightyCreak This should be ready for review. I have more changes locally (mostly to support https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/42432, but also some parsing stuff), but these should be simple/easy to review, so further changes make sense to put in a new PR later. |
|
I agree for the first two changes, but for the last one, the ordering was alphabetical (except for Intel drivers, which was a mistake on my part). The reason is that any other ordering will only be obvious to the person who is implementing it. I had that before and it was confusing in the end. After some pondering, using alphabetical ordering was the less confusing choice IMO. But maybe we can find a compromise for "Software" and "Translation", which are indeed not hardware vendors. How about "[Software]" and "[Translation]", ordered last in the list? |
|
I merged your first two fixes in #379 |
|
I'll open a fresh PR for the third change |
2 participants
A few updates.
Fix parsing of GL_ARB_multi_draw_indirect support
This line in features.txt currently has
supports, notsupport.(GL_ARB_texture_compression_bptc also uses
supports, but has other issues that break its parsing right now anyways.)I plan on also fixing these inconsistencies in upstream features.txt, but for the sake of the "date added" display, it's still beneficial to fix here as well.
Remove
nv30trackingThis tracking was misleading due to upstream only having marked 4 extensions as enabled for it. The last remnants of
nv30tracking were removed upstream with https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/42506Tweak displayed ordering of drivers