Enable faithful API for all ops (#47711)

smessmer · facebook-github-bot · commit 046ea6696dde · 2020-12-08T03:43:48.000-08:00
Summary: Pull Request resolved: #47711 Seems we generated the declaration but the definition only for c10-full ops. We should also generate the definition for non-c10-full ops. This makes future migrations of ops from non-c10-full to c10-full have a lower impact on the C++ API. ghstack-source-id: 118064755 Test Plan: waitforsandcastle Reviewed By: bhosmer Differential Revision: D24835006 fbshipit-source-id: 8f5c3c0ffcdc9b479ca3785d57da16db508795f5
diff --git a/tools/codegen/gen.py b/tools/codegen/gen.py
@@ -480,8 +480,7 @@ def generate_defn(sig: CppSignature) -> str:
 
         result = generate_defn(sig_group.signature)
         if sig_group.faithful_signature is not None:
-            if local.use_c10_dispatcher().dispatcher_uses_new_style():
-                result += generate_defn(sig_group.faithful_signature)
+            result += generate_defn(sig_group.faithful_signature)
 
         return result
 
