Deprecate bitbroadcast (#19771)

* Deprecate `bitbroadcast`

* Remove `bitbroadcast` docs

Author: pabloferz

base/broadcast.jl

@@ -7,8 +7,7 @@ using Base: promote_eltype_op, linearindices, tail, OneTo, to_shape,
             _msk_end, unsafe_bitgetindex, bitcache_chunks, bitcache_size, dumpbitcache,
             nullable_returntype, null_safe_eltype_op, hasvalue, is_nullable_array
 import Base: broadcast, broadcast!
-export bitbroadcast, dotview
-export broadcast_getindex, broadcast_setindex!
+export broadcast_getindex, broadcast_setindex!, dotview
 
 ## Broadcasting utilities ##
 # fallbacks for some special cases
@@ -453,24 +452,6 @@ julia> Ref(7) .+ Nullable(3)
 """
 @inline broadcast(f, A, Bs...) = broadcast_c(f, containertype(A, Bs...), A, Bs...)
 
-"""
-    bitbroadcast(f, As...)
-
-Like [`broadcast`](@ref), but allocates a `BitArray` to store the
-result, rather then an `Array`.
-
-```jldoctest
-julia> bitbroadcast(isodd,[1,2,3,4,5])
-5-element BitArray{1}:
-  true
- false
-  true
- false
-  true
-```
-"""
-@inline bitbroadcast(f, As...) = broadcast!(f, similar(BitArray, broadcast_indices(As...)), As...)
-
 """
     broadcast_getindex(A, inds...)
 

base/deprecated.jl

@@ -1346,6 +1346,9 @@ function quadgk(args...; kwargs...)
 end
 export quadgk
 
+# Broadcast now returns a BitArray when the resulting eltype is Bool (#17623)
+@deprecate bitbroadcast broadcast
+
 # Deprecate two-argument map! (map!(f, A)) for a cycle in anticipation of semantic change
 @deprecate map!{F}(f::F, A::AbstractArray) map!(f, A, A)
 @deprecate asyncmap!(f, c; ntasks=0, batch_size=nothing) asyncmap!(f, c, c; ntasks=ntasks, batch_size=batch_size)

base/exports.jl

@@ -463,7 +463,6 @@ export
     zeta,
 
 # arrays
-    bitbroadcast,
     broadcast!,
     broadcast,
     broadcast_getindex,

doc/src/stdlib/arrays.md

@@ -48,7 +48,6 @@ All mathematical operations and functions are supported for arrays
 ```@docs
 Base.broadcast
 Base.Broadcast.broadcast!
-Base.Broadcast.bitbroadcast
 ```
 
 ## Indexing, Assignment, and Concatenation

test/bitarray.jl

@@ -1136,8 +1136,8 @@ let b1 = trues(v1), b2 = falses(v1)
     end
 end
 
-let odds = bitbroadcast(isodd, 1:2000)
-    evens = bitbroadcast(iseven, 1:2000)
+let odds = broadcast(isodd, 1:2000)
+    evens = broadcast(iseven, 1:2000)
     for i = 1:2:2000
         @test findprev(odds,i)  == findprevnot(evens,i) == i
         @test findnext(odds,i)  == findnextnot(evens,i) == i