Skip to content

Commit

Permalink
halide: remove unnecessary workaround
Browse files Browse the repository at this point in the history
This was fixed in #112154.

Closes #112461.

Signed-off-by: Daniel Nachun <9205048+danielnachun@users.noreply.github.com>
Signed-off-by: BrewTestBot <1589480+BrewTestBot@users.noreply.github.com>
  • Loading branch information
carlocab authored and BrewTestBot committed Oct 6, 2022
1 parent 56fe10a commit 5630341
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions Formula/halide.rb
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,9 @@ class Halide < Formula
end

depends_on "cmake" => :build
depends_on "pybind11" => :build
depends_on "jpeg-turbo"
depends_on "libpng"
depends_on "pybind11"
depends_on "python@3.10"

fails_with gcc: "5" # LLVM is built with Homebrew GCC
Expand All @@ -44,10 +44,6 @@ def install
-DHalide_SHARED_LLVM=ON
-DPYBIND11_USE_FETCHCONTENT=OFF
]
llvm = deps.find { |dep| dep.name.match?(/^llvm(@\d+)?$/) }
.to_formula
# Apple libLTO cannot parse our object files.
args << "-DCMAKE_SHARED_LINKER_FLAGS=-Wl,-lto_library,#{llvm.opt_lib/shared_library("libLTO")}" if OS.mac?

system "cmake", "-S", ".", "-B", "build", *args, *std_cmake_args
system "cmake", "--build", "build"
Expand Down

0 comments on commit 5630341

Please sign in to comment.