im-lunex wrote: okay thanks for the review one quick question please.. we removed the check soo do we still need to have a test for specifically testing CallInst/InvokeInst ? cuz we the assertion entirely.. what do you suggest ? https://github.com/llvm/llvm-project/pull/221955