Skip to content

Commit

Permalink
re-instate the error which breaks Revise
Browse files Browse the repository at this point in the history
  • Loading branch information
mcabbott committed Aug 8, 2022
1 parent 85aa2fa commit b7e63f5
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/rulesets/Base/fastmath_able.jl
Original file line number Diff line number Diff line change
Expand Up @@ -302,11 +302,11 @@ let
non_transformed_definitions = intersect(fastable_ast.args, fast_ast.args)
filter!(expr->!(expr isa LineNumberNode), non_transformed_definitions)
if !isempty(non_transformed_definitions)
# error(
# "Non-FastMath compatible rules defined in fastmath_able.jl. \n Definitions:\n" *
# join(non_transformed_definitions, "\n")
# )
# This error() may not play well with Revise. But a wanring @error does:
error(
"Non-FastMath compatible rules defined in fastmath_able.jl. \n Definitions:\n" *
join(non_transformed_definitions, "\n")
)
# This error() may not play well with Revise. But a wanring @error does, we should change it:
@error "Non-FastMath compatible rules defined in fastmath_able.jl." non_transformed_definitions
end

Expand Down

0 comments on commit b7e63f5

Please sign in to comment.