Leo Famulari <l...@famulari.name> writes: > On Tue, Nov 08, 2016 at 09:10:46AM +0100, Ricardo Wurmus wrote: >> >> Leo Famulari <l...@famulari.name> 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