Skip to content

Commit

Permalink
fix wrong target names
Browse files Browse the repository at this point in the history
  • Loading branch information
yshekel committed Aug 21, 2024
1 parent d772b1b commit 747c798
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions icicle_v3/cmake/curve.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,9 @@ function(check_curve CURVE CURVE_INDEX_OUT FEATURES_STRING_OUT)
endfunction()


function(setup_curve_target CURVE_NAME CURVE_INDEX FEATURES_STRING)
function(setup_curve_target CURVE CURVE_INDEX FEATURES_STRING)
# the scalar field of the curve is built to a field library (like babybear is built)
setup_field_target(${CURVE_NAME} ${CURVE_INDEX} ${FEATURES_STRING})
setup_field_target(${CURVE} ${CURVE_INDEX} ${FEATURES_STRING})

add_library(icicle_curve SHARED)

Expand Down
4 changes: 2 additions & 2 deletions icicle_v3/cmake/field.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ function(check_field FIELD FIELD_INDEX_OUT FEATURES_STRING_OUT)
endif ()
endfunction()

function(setup_field_target FIELD_NAME FIELD_INDEX FEATURES_STRING)
function(setup_field_target FIELD FIELD_INDEX FEATURES_STRING)
add_library(icicle_field SHARED)

# Split FEATURES_STRING into a list using "," as the separator
Expand All @@ -56,7 +56,7 @@ function(setup_field_target FIELD_NAME FIELD_INDEX FEATURES_STRING)
target_link_libraries(icicle_field PUBLIC icicle_device pthread)

# Ensure FIELD is defined in the cache for backends to see
set(FIELD "${FIELD_NAME}" CACHE STRING "")
set(FIELD "${FIELD}" CACHE STRING "")
add_compile_definitions(FIELD=${FIELD} FIELD_ID=${FIELD_INDEX})

install(TARGETS icicle_field
Expand Down

0 comments on commit 747c798

Please sign in to comment.