in 46345:
Makes sense, I am thinking how to rename one of them now.
Also not clear to me why we don't just force all stores to use the version that does more checking...
It is possible but requires more refactoring changes, for example, emitInsStoreLcl and emitInsStoreInd should just receive GenTree* and calculate instruction ins, emitAttr attr inside. I will try to find time for this change later as a separate PR.
in 46345:
Makes sense, I am thinking how to rename one of them now.
Also not clear to me why we don't just force all stores to use the version that does more checking...
It is possible but requires more refactoring changes, for example, emitInsStoreLcl and emitInsStoreInd should just receive GenTree* and calculate instruction ins, emitAttr attr inside. I will try to find time for this change later as a separate PR.