[PATCH] Drop the udis86 build dependency

Alp Toker alp at nuanti.com
Thu Jul 17 13:14:43 PDT 2014


Thanks guys, removed in r213300.

On 17/07/2014 22:06, Eric Christopher wrote:
> Cool, thanks Anton.
>
> -eric
>
> On Thu, Jul 17, 2014 at 12:03 PM, Anton Korobeynikov
> <anton at korobeynikov.info> wrote:
>> I don't recall all the details, but iirc the rationale was to do
>> something in order to debug the old JIT easier. I think it can be
>> simply dropped now.
>>
>> On Thu, Jul 17, 2014 at 9:32 PM, Alp Toker <alp at nuanti.com> wrote:
>>> On 17/07/2014 20:30, Eric Christopher wrote:
>>>> Did you try to ping the original committer for this to check in about it?
>>>
>>> CC'ing in Anton who added it in 2007.
>>>
>>>> -eric
>>>>
>>>>
>>>> On Thu, Jul 17, 2014 at 6:59 AM, Alp Toker <alp at nuanti.com> wrote:
>>>>> There's no need to incur a user-facing LLVM build dependency on the
>>>>> external
>>>>> udis86 project just to support a single debug print statement.
>>>>>
>>>>> The machine code is printed by LLVM and can be easily piped into udis86
>>>>> (or
>>>>> indeed LLVM's own disassembler) to provide the same debugging experience
>>>>> without having to link a third-party disassembler into LLVM binaries.
>>>>>
>>>>> --
>>>>> http://www.nuanti.com
>>>>> the browser experts
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> llvm-commits mailing list
>>>>> llvm-commits at cs.uiuc.edu
>>>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>>>>
>>> --
>>> http://www.nuanti.com
>>> the browser experts
>>>
>>
>>
>> --
>> With best regards, Anton Korobeynikov
>> Faculty of Mathematics and Mechanics, Saint Petersburg State University

-- 
http://www.nuanti.com
the browser experts




More information about the llvm-commits mailing list