{"push_id":641611412,"size":519,"distinct_size":3,"ref":"refs/heads/fixes","head":"1ec2e361b281020a682f4a1bfc5368392a318fd9","before":"f497ff6ef648528dd8d8aa20b90b600f7df07497","commits":[{"sha":"91e3533481d6921c4b46109742d6f67b7f897f86","author":{"email":"f2295d84e358395675bc8031be58672073ae065e@redhat.com","name":"Dave Airlie"},"message":"st_glsl_to_tgsi: only do mov copy propagation on temps (v2)\n\nDon't propagate ARRAYs\n\nThis should fix:\nhttps://bugs.freedesktop.org/show_bug.cgi?id=89759\n\nv2: just specify arrays so we get input propagation\nSigned-off-by: Dave Airlie <airlied@redhat.com>\nCc: mesa-stable@lists.freedesktop.org\nReviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/91e3533481d6921c4b46109742d6f67b7f897f86"},{"sha":"9d1b5febb62d74c9fc564635d4e0fa5207928c46","author":{"email":"bcbbb7aa705ec3a5fc5824c01a845186c3d62fca@alum.mit.edu","name":"Ilia Mirkin"},"message":"st/mesa: update arrays when the current attrib has been updated\n\nFixes the recently-sent gl-2.0-vertex-const-attr piglit test. Makes sure\nto revalidate arrays when only the current attribute has been updated\nvia glVertexAttrib*.\n\nBugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89754\nSigned-off-by: Ilia Mirkin <imirkin@alum.mit.edu>\nReviewed-by: Marek Olšák <marek.olsak@amd.com>\nCc: \"10.4 10.5\" <mesa-stable@lists.freedesktop.org>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/9d1b5febb62d74c9fc564635d4e0fa5207928c46"},{"sha":"2e34faaf06f50741164209cc5743ab4d8579c94c","author":{"email":"bcbbb7aa705ec3a5fc5824c01a845186c3d62fca@alum.mit.edu","name":"Ilia Mirkin"},"message":"gallium/hud: avoid overflowing hud graph name size\n\nSpotted by Coverity.\n\nSigned-off-by: Ilia Mirkin <imirkin@alum.mit.edu>\nReviewed-by: Marek Olšák <marek.olsak@amd.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/2e34faaf06f50741164209cc5743ab4d8579c94c"},{"sha":"1b87d73a9f93fd0ce2c943964de9eae44affb991","author":{"email":"bcbbb7aa705ec3a5fc5824c01a845186c3d62fca@alum.mit.edu","name":"Ilia Mirkin"},"message":"gallium/util: remove u_linkage\n\nDoes not appear to be used in tree. Coverity spotted some errors in the\nbitmask stuff, but the whole thing appears to be unused.\n\nSigned-off-by: Ilia Mirkin <imirkin@alum.mit.edu>\nReviewed-by: Brian Paul <brianp@vmware.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/1b87d73a9f93fd0ce2c943964de9eae44affb991"},{"sha":"f95a6b2ff4237181fd7e9622e19e9aa0719a1f21","author":{"email":"bcbbb7aa705ec3a5fc5824c01a845186c3d62fca@alum.mit.edu","name":"Ilia Mirkin"},"message":"st/mesa: initialize have_fma in constructor\n\nSpotted by Coverity.\n\nSigned-off-by: Ilia Mirkin <imirkin@alum.mit.edu>\nReviewed-by: Brian Paul <brianp@vmware.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/f95a6b2ff4237181fd7e9622e19e9aa0719a1f21"},{"sha":"626434893a048f3ba89a751998fcb3789a3dbb96","author":{"email":"bcbbb7aa705ec3a5fc5824c01a845186c3d62fca@alum.mit.edu","name":"Ilia Mirkin"},"message":"tgsi: fix out-of-bounds access for cube arrays\n\nThe CUBE_ARRAY case uses r[4]. Make sure that the stack variable is\nthere.\n\nNoticed by Coverity.\n\nSigned-off-by: Ilia Mirkin <imirkin@alum.mit.edu>\nReviewed-by: Roland Scheidegger <sroland@vmware.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/626434893a048f3ba89a751998fcb3789a3dbb96"},{"sha":"787aa26cb7b48504f7770cacfc321324ecafa29a","author":{"email":"c7ea837d7a46effe4232b086213468b8b31643bf@amd.com","name":"Christian König"},"message":"gallium/vl: partially revert \"Use util_cpu_to_le{16,32} in many more places.\"\n\nThe data in memory is in big endian format and needs to be converted\ninto CPU byte order. So the patch actually reversed what needs to be done.\n\nSigned-off-by: Christian König <christian.koenig@amd.com>\nReviewed-by: Matt Turner <mattst88@gmail.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/787aa26cb7b48504f7770cacfc321324ecafa29a"},{"sha":"8dad9455ff748c543635b24908566c3b94cb93a9","author":{"email":"7d58aee419d6f9201d75517c885374dff5e7d848@vmware.com","name":"Roland Scheidegger"},"message":"gallivm: pass jit_context pointer through to sampling\n\nThe callbacks used for getting the dynamic texture/sampler state were using\nthe jit_context from the generated jit function. This works just fine, however\nthat way it's impossible to generate separate functions for texture sampling,\nas will be done in the next commit. Hence, pass this pointer through all\ninterfaces so it can be passed to a separate function (technically, it would\nprobably be possible to extract this pointer from the current function instead,\nbut this feels hacky and would probably require some more hacks if we'd use\nreal functions instead of inlining all shader functions at some point).\nThere should be no difference in the generated code for now.\n\nReviewed-by: Jose Fonseca <jfonseca@vmware.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/8dad9455ff748c543635b24908566c3b94cb93a9"},{"sha":"56076be2ac60f005770516a9ac305e60e6285497","author":{"email":"7d58aee419d6f9201d75517c885374dff5e7d848@vmware.com","name":"Roland Scheidegger"},"message":"gallivm: use llvm function calls for texturing instead of inlining\n\nThere are issues with inlining everything, most notably llvm will use much\nmore memory (and be slower) when compiling. Ideally we'd probably use\nfunctions for shader functions too but texture sampling usually is responsible\nfor quite some IR (it can easily reach 80% of total IR instructions) so this\nseems like a good start.\nThis still generates a different function for all different combinations just\nlike before, however it is possible llvm is missing some optimization\nopportunities - it is believed though such opportunities should be somewhat\nrare, but at least for now it can still be switched off (at compile time only).\nIt should probably make compiled code also smaller because the same function\nshould be used for different variants in the same module (so for the\nopaque/partial or linear/elts variants).\nNo piglit change (though it does indeed speed up unrealistic tests like\nfp-indirections2 by a factor of 30 or so).\nHas a small negative performance impact in openarena - I suspect this could\nbe fixed by running some IPO passes (despite the private linkage, llvm right\nnow does NO optimization at all wrt anything going past the call, even if\nthere's just one caller - so things like values stored before the call and then\nalways written by the function etc. will not be optimized away, nor will dead\narguments (which we mostly shouldn't have) be eliminated, always constant\narguments promoted etc.).\n\nv2: use proper return values instead of pointer function arguments.\nllvm supports aggregate return values, which do wonders here eliminating\nunnecessary stack variables - everything in fact will be returned in registers\neven without any IPO optimizations. It makes the code simpler too.\nWith this I could not measure a peformance impact in openarena any longer\n(though since there's still no constant value propagation etc. into the tex\nfunctions this does not mean it couldn't have a negative impact elsewhere).\n\nv3: fix some minor issues suggested by Jose, and do disassembly (and the\nprofiling) without hacks.\n\nReviewed-by: Jose Fonseca <jfonseca@vmware.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/56076be2ac60f005770516a9ac305e60e6285497"},{"sha":"afa9fc15613c3ebdc59245c9d4d8c1280b2a2a37","author":{"email":"96f164ad4d9b2b0dacf8ebee2bb1eeb3aa69adf1@anholt.net","name":"Eric Anholt"},"message":"nir: Add optional lowering of flrp.\n\nReviewed-by: Matt Turner <mattst88@gmail.com>\nReviewed-by: Connor Abbott <cwabbott0@gmail.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/afa9fc15613c3ebdc59245c9d4d8c1280b2a2a37"},{"sha":"a99a16a0cffbb79e1f9a2d2844c549e6ff91afdb","author":{"email":"ed664c4bb5cadd95cb7521d1190a56e280fa6954@rutgers.edu","name":"Jan Vesely"},"message":"gallivm: Fix build since llvm r233411\n\nSigned-off-by: Jan Vesely <jan.vesely@rutgers.edu>\nReviewed-by: Tom Stellard <thomas.stellard@amd.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/a99a16a0cffbb79e1f9a2d2844c549e6ff91afdb"},{"sha":"49b86007aa2bb599ada6cdbed7ff56246917f12e","author":{"email":"bcbbb7aa705ec3a5fc5824c01a845186c3d62fca@alum.mit.edu","name":"Ilia Mirkin"},"message":"nv50/ir: take postFactor into account when doing peephole optimizations\n\nMultiply operations can have a post-factor on them, which other ops\ndon't support. Only perform the peephole optimizations when there is no\npost-factor involved.\n\nBugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89758\nCc: \"10.4 10.5\" <mesa-stable@lists.freedesktop.org>\nSigned-off-by: Ilia Mirkin <imirkin@alum.mit.edu>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/49b86007aa2bb599ada6cdbed7ff56246917f12e"},{"sha":"58030a8f99d94d6c1bab02ef113d93c6c2636216","author":{"email":"bcbbb7aa705ec3a5fc5824c01a845186c3d62fca@alum.mit.edu","name":"Ilia Mirkin"},"message":"nv50/ir/gk110: fix offset flag position for TXD opcode\n\nCc: \"10.4 10.5\" <mesa-stable@lists.freedesktop.org>\nSigned-off-by: Ilia Mirkin <imirkin@alum.mit.edu>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/58030a8f99d94d6c1bab02ef113d93c6c2636216"},{"sha":"2cb149c2894679c5c7af81b40277f77cc0fb099d","author":{"email":"7f85c8176b2576ffafd04ac63cd29ed35ff86cb6@yahoo.com.au","name":"Timothy Arceri"},"message":"glsl: mark uniform and input interface blocks as read only\n\nReviewed-by: Mark Janes <mark.a.janes@intel.com>\nReviewed-by: Kenneth Graunke <kenneth@whitecape.org>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/2cb149c2894679c5c7af81b40277f77cc0fb099d"},{"sha":"764fc2be5ab61c86baaab746e7ea7702328c4f9f","author":{"email":"7d58aee419d6f9201d75517c885374dff5e7d848@vmware.com","name":"Roland Scheidegger"},"message":"gallivm: fix texture function name (key) when using txf/ld\n\nWhen using the texel fetch functions rather than ordinary texturing,\nthe arguments are all int vecs instead of float vecs, not to mention\nthe actual function would look completely different. Hence this must\nbe included in the texture function name (which serves as the key)\notherwise things crash badly when a shader accesses the same texture\nand sampler unit with both txf/ld and ordinary texturing instructions\nwith otherwise matching keys.","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/764fc2be5ab61c86baaab746e7ea7702328c4f9f"},{"sha":"b2424fb0304cf4afd363b35c1dab49fb7edddb08","author":{"email":"7d58aee419d6f9201d75517c885374dff5e7d848@vmware.com","name":"Roland Scheidegger"},"message":"llvmpipe: simplify address calculation for 4x4 blocks\n\nThese functions looked quite complicated, even though what they actually did\nwas trivial (ever since we dropped swizzled rendering). Also drop lookup of\nformat block per bytes done for each block, and do it once per scene instead.\nThis improves everybody's favorite \"benchmark\" by 3% or so, though\nlp_rast_shade_quads_all() which calls this shows up still quite high for a\nfunction which does little more than call the jit function.\n(This would most likely be much better handled by the jit function itself,\nthe strides are passed through anyway already, though for being able to\nhandle layers it would definitely add some complexity.)\n\nReviewed-by: Jose Fonseca <jfonseca@vmware.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/b2424fb0304cf4afd363b35c1dab49fb7edddb08"},{"sha":"9d32d358500733249d3c0264c7458c2e5a65f515","author":{"email":"cf148a1c890adeeb4308e9e093ed95f3d05c8318@intel.com","name":"Ben Widawsky"},"message":"i965/skl: Disable partial resolve in VC\n\nRecomendation [sic] is to set this field to 1 always. Programming it to default\nvalue of 0, may have -ve impact on performance for MSAA WLs.\n\nAnother don't suck bit which needs to get set.\n\nThe patch wasn't as well tested as I would have liked, primarily I don't have\nperf numbers for it, but it's getting to a point where it is in danger of being\nlost.\n\nv2: v1 was a mix of two patches. Since 0x7004 is masked, we only need to set it\nonce at initialization and make sure the pma workaround doesn't set the mask bit\n(which it doesn't).\nMove LRI to init gpu state (Ken)\nAdd a comment.\n\nSigned-off-by: Ben Widawsky <ben@bwidawsk.net>\nReviewed-by: Kenneth Graunke <kenneth@whitecape.org>\nReviewed-by: Anuj Phogat <anuj.phogat@gmail.com>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/9d32d358500733249d3c0264c7458c2e5a65f515"},{"sha":"74fd226e34d0cf5e9ff43174ae69b4a66f5de1ab","author":{"email":"cf148a1c890adeeb4308e9e093ed95f3d05c8318@intel.com","name":"Ben Widawsky"},"message":"i965/skl: Don't use the PMA depth stall workaround\n\nThe PMA depth stall must be enabled (optimization turned off) under certain\ncircumstances on gen8. This was supposedly fixed for Gen9, which means we do not\nneed to check, or toggle the state. The hardware is supposed to enable the\nhardware optimization by default, unlike BDW, so we also don't need to set it at\ninit. For whatever reason this improves stability on ETQW with the bug mentioned\nbelow.\n\nReferences: https://bugs.freedesktop.org/show_bug.cgi?id=89039 (doesn't fix)\nSigned-off-by: Ben Widawsky <ben@bwidawsk.net>\nTested-by: Anuj Phogat <anuj.phogat@intel.com>\nReviewed-by: Kenneth Graunke <kenneth@whitecape.org>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/74fd226e34d0cf5e9ff43174ae69b4a66f5de1ab"},{"sha":"75c922e0fed41acf28318b082f7f915c752ffe74","author":{"email":"bd2562f754ec92342f93f61c25d731e290a2ffa8@whitecape.org","name":"Kenneth Graunke"},"message":"nir: Add nir_builder helpers for creating load_const intrinsics.\n\nBoth prog->nir and tgsi->nir will want to use these.\n\nSigned-off-by: Kenneth Graunke <kenneth@whitecape.org>\nReviewed-by: Connor Abbott <cwabbott0@gmail.com>\nReviewed-by: Eric Anholt <eric@anholt.net>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/75c922e0fed41acf28318b082f7f915c752ffe74"},{"sha":"06f7bea96ab62e7de1523de837f5b69c1bac6513","author":{"email":"bd2562f754ec92342f93f61c25d731e290a2ffa8@whitecape.org","name":"Kenneth Graunke"},"message":"nir: Add builder helpers for MOVs with ALU sources and swizzling MOVs.\n\nThese will be useful for prog->nir and tgsi->nir.\n\nv2: Don't forget to mark nir_swizzle as inline (Eric).\n\nSigned-off-by: Kenneth Graunke <kenneth@whitecape.org>\nReviewed-by: Connor Abbott <cwabbott0@gmail.com>\nReviewed-by: Eric Anholt <eric@anholt.net>","distinct":false,"url":"https://api.github.com/repos/jonathangray/mesa/commits/06f7bea96ab62e7de1523de837f5b69c1bac6513"}]}