[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.
|
From: |
Ricardo Wurmus |
|
Subject: |
Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support. |
|
Date: |
Tue, 08 Nov 2016 21:02:00 +0100 |
|
User-agent: |
mu4e 0.9.16; emacs 26.0.50.1 |
Leo Famulari <address@hidden> writes:
> On Tue, Nov 08, 2016 at 09:10:46AM +0100, Ricardo Wurmus wrote:
>>
>> Leo Famulari <address@hidden> writes:
>>
>> > On Sun, Nov 06, 2016 at 08:34:23PM +0100, Ricardo Wurmus wrote:
>> >> * gnu/packages/llvm.scm (llvm-with-rtti): New variable.
>> >
>> > Seems okay, although would it make sense to enable this in the main LLVM
>> > package?
>>
>> I’m not sure, because it seems to require dependent packages to also
>> support this or else be built with “-fno-rtti”. We should build this
>> out in a separate branch and see how much damage this causes and how
>> many changes to existing packages this requires.
>
> Perhaps, but let's not let it delay this patch :)
Okay :) Pushed as 8ae60404510fedb89502e27c0e73f830ab429641.
Thanks for the review!
~~ Ricardo