On 14/02/17 04:50 AM, Matt Arsenault wrote: >> On Feb 6, 2017, at 19:42, Timothy Arceri <tarc...@itsqueeze.com >> <mailto:tarc...@itsqueeze.com>> wrote: >> >> This series does not include the patch that adds cache support >> to the radeonsi backend, the main reason for this is that llvm >> currently doesn't allow the version to be queried at runtime >> (as far as I'm aware) although it seems like other are interested >> in this feature [1] so I will follow up on that. > > This should not be necessary. Old bitcode is supposed to be forward > compatible and should be autoupgraded. You should not need to check that > the runtime version matches.
This isn't enough for the shader cache, since LLVM code changes can potentially cause different machine code to be generated even for identical LLVM IR, e.g. a fix for a bug which caused incorrect machine code to be generated. -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
