guix-devel
[Top][All Lists]
Advanced

[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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]