circt icon indicating copy to clipboard operation
circt copied to clipboard

[SV] BindOp::verifySybmolUses could be expensive

Open uenoku opened this issue 1 year ago • 0 comments

BindOp refers an instance through an inner symbol and BindOp::verifySymbolUses>findInstanceSymbolInBlock looks up an instance by walking entire modules in the worst case. It hasn't cause any issue probably because bind ops are created in the later pipeline.

https://github.com/llvm/circt/blob/695e36e5d480780f830bf2ded05c3fb8c8eb6012/lib/Dialect/SV/SVOps.cpp#L1864

uenoku avatar May 28 '24 08:05 uenoku