[Provisioner] Remove NETWORK_PADDING_FACTOR

pytorch · Apr 16, 2019 · 46e177e · 46e177e
1 parent fd75ae5
commit 46e177e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/include/glow/Runtime/Provisioner/Provisioner.h b/include/glow/Runtime/Provisioner/Provisioner.h
@@ -51,10 +51,6 @@ class Provisioner final {
   /// functions.
   std::unordered_map<std::string, std::unique_ptr<CompiledFunction>> functions_;
 
-  /// Padding factor to account for generated code size. Should be greater
-  /// than 1.0.
-  const float NETWORK_PADDING_FACTOR = 1.1;
-
   /// List of available DeviceManagers added during initialization.
   std::vector<DeviceManager *> devices_;
 };

diff --git a/lib/Runtime/Provisioner/Provisioner.cpp b/lib/Runtime/Provisioner/Provisioner.cpp
@@ -102,8 +102,7 @@ llvm::Error Provisioner::provision(DAGListTy &networks, Module &module) {
 
   // Try to add functions to devices in order from largest to smallest.
   for (unsigned i = 0; i < logicalDeviceSize.size(); i++) {
-    RETURN_ERR_IF_NOT(logicalDeviceSize[i].second * NETWORK_PADDING_FACTOR <
-                          deviceMemory[i].second,
+    RETURN_ERR_IF_NOT(logicalDeviceSize[i].second < deviceMemory[i].second,
                       "Not enough memory to provision functions onto devices");
 
     // Load functions on device.