add_visited_pcs should not be part of State trait

[tdelabro]

blockifier/crates/blockifier/src/state/state_api.rs

Lines 106 to 109 in 1f54a54

	/// Marks the given set of PC values as visited for the given class hash.
	// TODO(lior): Once we have a BlockResources object, move this logic there. Make sure reverted
	// entry points do not affect the final set of PCs.
	fn add_visited_pcs(&mut self, class_hash: ClassHash, pcs: &HashSet<usize>);

As explained in the comment this method should not be part of the State trait.
It makes it weird to implement for any user of the lib. They now have to mirror your architecture otherwise they can't correctly impl the trait.

Can you please remove it from State and put it in another trait that may, or may not be implemented by the same structure?