opal icon indicating copy to clipboard operation
opal copied to clipboard

Add test setup for alias analysis

Open LukasKlenner opened this issue 1 year ago • 5 comments

This PR adds a setup for testing alias analyses, including some (simple) test cases.

More (complex) test cases alongside analyses that pass them will be added in future PR's. This is a continuation of #171

LukasKlenner avatar Jan 08 '24 23:01 LukasKlenner

Build doesn't work for several reasons, please fix

errt avatar Jan 09 '24 09:01 errt

building issues are fixed, but it fails because the AliasTests don't pass, despite them being annotated with @Ignored. Is it not supported or is something else required to make the tests not being executed?

LukasKlenner avatar Jan 09 '24 14:01 LukasKlenner

PS: AliasEntity and AliasSourceElement need to be in the tac package (and not br) because they rely on the DefinitionSite class

LukasKlenner avatar Jan 10 '24 10:01 LukasKlenner

PS: AliasEntity and AliasSourceElement need to be in the tac package (and not br) because they rely on the DefinitionSite class

This may hint at potential to improve. In many cases, we map TAC use sites to bytecode program counters and values (search for persistentUVar), a similar mapping, maybe even excluding the values, might be applicable here.

errt avatar Jan 11 '24 12:01 errt

All previous issues have been addressed, but most of the classes have been completely reworked

LukasKlenner avatar Mar 19 '24 14:03 LukasKlenner