GitLab

Doing this allows for things to be university across all our targets without neededing an op for each ref.

TODO. breaks right now. Reusing tests from previous iteration with some tweaks. Need to get back to this once I get better train of thought

Conversion works, just issues with predicate right now.

I may need additional checks for this, or to somehow find the matching division
by zero, and cause a dangling reference so this gets flagged correctly at compile time.
Current attempt inserts a divzero instruction that would later get picked up at the verify stage
during compile. Not sure if this is correct incase we run into operator collisions down the road

This reverts commit fcc84214.

This reverts commit 4aeacc17.

Use this call to also skip converts when running a simplify_algebra pass over a program.

Making things more maintainable by splitting the unit tests that shared the baseline program for validation which resulted in an & case that was a bit more cumbersome to debug.

Allows us to not throw warnings instead of using [[maybe_unused]] flag instead.

Adds this to handle broadcasted values instead of just scalars

Used to avoid the case where  1e-12 is used and is matched as zero errornously resulting in removing the call with the incorrect value.

Throw an exception when this occurs to indicate our simpliciation passes resulted in a singularity somewhere. Related to #1236

Simplify addition zero multiplication and divide operations. Added approrpiate test cases with returns and replacing the instruction and operand to just return zero.

Using the unit/neg unit matchers to handle subtraction operations in the same steps. Added unit tests for both cases.

Part of changes that go wtih #1236. Reverts -1 divide operations to a simple negation of the parameter

Add handling for zero addition operations into the find_unit_ops() matcher functor.

Added test case and code to simplify zero additions between paremeters and literals during simplifications.  In reference to issue #1236

Simplfies our code for all operations and reusing original unit tests for overalpping matcher.

Done to satisfy simplifications specified by #1236 .  Just replace every  parameter divided by 1 with itself. It's assumed that the eliminate_identity() pass will handle generated identity operators in our run_pass()

Save a multiply operation with that of a negation of  input parameter x. Suggested improvement via #1236

Original use case of having a literal 1, instead of any other number in simplify_mul_add, resulted in the find_unit_mult_const function to optimize away the literal 1 causing this test to fail. on the final check. Switched the constant to a non zero & one value, and now correctly passes.

Commit for the day, work in progress as I'm failing one of our unit tests outside of the change

This is an extension to insert_module_instructions, but instead of just inserting from a module, it can insert a range or a vector of instructions.

should be --fp16 , not --fp16ref

Allows PyTorch converted version of SSD-resnet34 to work

 Compiles significantly faster than constructing all the objects. It also reduces recompiles as well.