From 46e177e3acc8e91932ff6516cbc8b3d8f12b9426 Mon Sep 17 00:00:00 2001
From: Man Wang <wangm@fb.com>
Date: Tue, 16 Apr 2019 11:09:19 -0700
Subject: [PATCH] [Provisioner] Remove NETWORK_PADDING_FACTOR

---
 include/glow/Runtime/Provisioner/Provisioner.h | 4 ----
 lib/Runtime/Provisioner/Provisioner.cpp        | 3 +--
 2 files changed, 1 insertion(+), 6 deletions(-)

diff --git a/include/glow/Runtime/Provisioner/Provisioner.h b/include/glow/Runtime/Provisioner/Provisioner.h
index 79e4078173..eeecc01dd5 100644
--- 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
index 144f014494..93f7dc024f 100644
--- 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.