[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#55057: [PATCH core-updates] gnu: llvm-14, llvm-12: Enable RTTI.
From: |
Marius Bakke |
Subject: |
bug#55057: [PATCH core-updates] gnu: llvm-14, llvm-12: Enable RTTI. |
Date: |
Thu, 01 Sep 2022 23:05:02 +0200 |
Greg Hogan <code@greghogan.com> skriver:
> The llvm packages currently define LLVM_REQUIRES_RTTI rather than the
> LLVM_ENABLE_RTTI flag specified in the LLVM documentation (
> https://llvm.org/docs/CMake.html).
>
> LLVM_REQUIRES_RTTI is noted as an internal flag in the llvm code (
> https://github.com/llvm/llvm-project/blob/llvmorg-14.0.1/llvm/cmake/modules/AddLLVM.cmake#L15
> ).
>
> This appears to have been an issue from the first Guix commit adding the
> LLVM_REQUIRES_RTTI flag as LLVM_ENABLE_RTTI is the flag specified in the
> documentation for llvm 3.8.1 (
> https://releases.llvm.org/3.8.1/docs/CMake.html).
> $ git show 83c49858b518b98f88db5f50ce36c19084e7ad62:gnu/packages/llvm.scm
Good catch. I wonder how we did not notice, apparently not many
packages depend on LLVM RTTI after all.
>>From 0cd3048f15fcca774088ab4d5b97a0fdaa74652e Mon Sep 17 00:00:00 2001
> From: Greg Hogan <code@greghogan.com>
> Date: Thu, 21 Apr 2022 19:13:50 +0000
> Subject: [PATCH] gnu: llvm-14, llvm-12: Enable RTTI.
>
> * gnu/packages/llvm.scm (llvm-14, llvm-12): Enable RTTI.
> [arguments]<#:configure-flags>: Switch to proper CMake flag.
Applied, thanks!
signature.asc
Description: PGP signature
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- bug#55057: [PATCH core-updates] gnu: llvm-14, llvm-12: Enable RTTI.,
Marius Bakke <=