iree-org / iree-jax

Apache License 2.0
48 stars 19 forks source link

Fix verification error when importing duplicate JAX primitives. #8

Closed stellaraccident closed 2 years ago

stellaraccident commented 2 years ago

We were using a prior iteration of the SymbolTable API, which lacked features for more robust renaming (which landed upstream but were not incorporated). This patch now uses the module name (which is descriptive for JAX exported modules) to prefix all functions and suffixes them further if still not unique.

Also does verification and error reporting before attempting to compile, resulting in better error messages.