tf-encrypted / moose

Secure distributed dataflow framework for encrypted machine learning and data processing
Apache License 2.0
58 stars 16 forks source link

Minor changes from direct execution exploration #1039

Closed mortendahl closed 2 years ago

mortendahl commented 2 years ago

This PR also contains (reverted) commits from running direct execution exploration. Bottom line however is that these did not result in any noticeable performance differences.

As a result, this PR only contributes minor changes that came up during the exploration, including a convenient DispatchKernel::execute function that defaults to calling DispatchKernel::compile immediately followed by calling the resulting kernel.

mortendahl commented 2 years ago

The only issue that I see is why do we have a file with 33k lines in moose?

It's been used by one of the benchmarks for a while (but had to fix it to match recent op and type renaming)