zhangyiss/mlx
1
0
Fork 0
mirror of https://github.com/ml-explore/mlx.git synced 2025-08-16 06:56:46 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

simplify

Browse Source
This commit is contained in:
Awni Hannun 2024-01-15 21:37:31 -08:00
parent b75ff47098
commit d34d10ebac
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
mlx/compile.cpp
View File

@ -1,5 +1,4 @@
// Copyright © 2023 Apple Inc.
#include <map>
#include <unordered_map>
#include <unordered_set>
@ -252,7 +251,6 @@ void compile_simplify(
for (auto& arr : tape) {
// Helper to check if we can fuse the parents of the
// given array
// If an array has no parents and siblings have
auto maybe_fuse_parents = [&](auto& a) {
auto parents = parents_map.find(a.id());
if (parents != parents_map.end()) {
@ -276,7 +274,7 @@ void compile_simplify(
}
return false;
} else {
return output_set.find(a.id()) != output_set.end();
return output_set.find(a.id()) == output_set.end();
}
};
@ -349,8 +347,7 @@ std::function<std::vector<array>(const std::vector<array>&)> compile(
compile_dfs(entry.inputs, entry.outputs);
// Simplify the tape
// compile_simplify(entry.tape, parents_map, entry.outputs, /* passes */
// 2);
compile_simplify(entry.tape, parents_map, entry.outputs, /* passes */ 2);
// This is a good point to do more optimizations, e.g. kernel fusion to
// generate new primitives. The tape needs to be updated accordingly