Skip to content

Commit

Permalink
Disable -fno-strict-float-cast-overflow flag
Browse files Browse the repository at this point in the history
Summary:
This breaks the open source iOS build, back it out temporarily since it
doesn't have the intended effect anyway.

Reviewed By: mhorowitz

Differential Revision: D33290406

fbshipit-source-id: b5dbfa93a1a04e1e1b53d48b857cba867393eaaf
  • Loading branch information
neildhar authored and facebook-github-bot committed Jan 5, 2022
1 parent cefe839 commit 20aa6fa
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion cmake/modules/Hermes.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -420,7 +420,9 @@ if (GCC_COMPATIBLE)
add_flag_if_supported("-Wdelete-non-virtual-dtor" DELETE_NON_VIRTUAL_DTOR_FLAG)

# Avoid triggering arbitrary UB when converting doubles to ints.
add_flag_if_supported("-fno-strict-float-cast-overflow" NO_STRICT_FLOAT_CAST_OVERFLOW_FLAG)
# TODO(T108716033) Evaluate adding this flag back in once a new clang is
# released or XCode is fixed.
# add_flag_if_supported("-fno-strict-float-cast-overflow" NO_STRICT_FLOAT_CAST_OVERFLOW_FLAG)

# Disable range loop analysis warnings.
check_cxx_compiler_flag("-Wrange-loop-analysis" RANGE_ANALYSIS_FLAG)
Expand Down

0 comments on commit 20aa6fa

Please sign in to comment.