On 01.04.2015 00:36, Roland Scheidegger wrote:
> Am 31.03.2015 um 08:33 schrieb Michel Dänzer:
>> From: Michel Dänzer <michel.daen...@amd.com>
>>
>> Signed-off-by: Michel Dänzer <michel.daen...@amd.com>
>> ---
>> src/gallium/auxiliary/gallivm/lp_bld_debug.cpp | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/src/gallium/auxiliary/gallivm/lp_bld_debug.cpp
>> b/src/gallium/auxiliary/gallivm/lp_bld_debug.cpp
>> index d4d453d..65d2896 100644
>> --- a/src/gallium/auxiliary/gallivm/lp_bld_debug.cpp
>> +++ b/src/gallium/auxiliary/gallivm/lp_bld_debug.cpp
>> @@ -256,8 +256,13 @@ disassemble(const void* func, llvm::raw_ostream & Out)
>> }
>>
>>
>> +#if HAVE_LLVM >= 0x0307
>> + OwningPtr<MCInstPrinter> Printer(
>> + T->createMCInstPrinter(llvm::Triple(Triple), AsmPrinterVariant,
>> *AsmInfo, *MII, *MRI));
>> +#else
>> OwningPtr<MCInstPrinter> Printer(
>> T->createMCInstPrinter(AsmPrinterVariant, *AsmInfo, *MII, *MRI,
>> *STI));
>> +#endif
>> if (!Printer) {
>> Out << "error: no instruction printer for target " << Triple.c_str()
>> << "\n";
>> Out.flush();
>>
>
> Does it actually work? I recently tried to do disassemble (more by
> accident, actually) with llvm 3.6 and it didn't work at all.
TBH I didn't test it, but I can see that it doesn't seem to work in any
useful way. Anyway, this change doesn't have any effect with LLVM 3.6,
so it sounds like it was already broken before.
> Anyway,
>
> Reviewed-by: Roland Scheidegger <srol...@vmware.com>
Thanks.
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
