Sarebbe il caso di pulire il codice e dichiarare le variabili, l'uso di Option Explicit non è affatto una brutta abitudine, anzi:
Private Sub cmdCinema_Click()
Dim riga As Long, indi As Long
If Txtcinema.Text = "" Then
MsgBox "Inserire dati per la ricerca!"
Exit Sub
End If
ActiveSheet.UsedRange.Select ' questa potresti evitarla
riga = Worksheets(1).UsedRange.Rows.Count ' anche questa
' UsedRange ha il difetto di non definire sempre il range realmente usato
' inoltre assegnare a riga l'ultima, ha poco senso in quanto la usi una sola volta
' prova per curiosità l'istruzione seguente che ti ho messo nel ciclo For
For indi = 2 To cells(Rows.Count, 1).End(xlUp).Row
If UCase(Cells(indi, 2).Value) <> UCase(Txtcinema.Text) Then Range("B" & indi).EntireRow.Hidden = True
Next
End Sub
Oltre ai commenti nel codice, ti ho anche evidenziato due modi per usare la cella, sia con Cells che con Range, perché nel caso di colonne numeriche, la prima ti tornerà più utile.
Ciao.