Skip to content

Commit 91bf785

Browse files
angelayifacebook-github-bot
authored andcommitted
Change docs about kwargs
Summary: I removed the statement that kwargs do not exist in the ATen dialect. Look at T160625628 for more context. Reviewed By: kimishpatel, larryliu0820 Differential Revision: D48484077 fbshipit-source-id: 997bb76f085404fcf94995599e76d97ccb29bbd4
1 parent 8043bc1 commit 91bf785

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

docs/website/docs/ir_spec/00_exir.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -213,9 +213,10 @@ A `call_function` node represents a call to an operator.
213213
1. In FX graph, a call_function can refer to any callable, in EXIR, we restrict
214214
this to only Canonical ATen operators (a select subset of PyTorch ATen operator
215215
library), custom operators and control flow operators.
216-
2. In FX graph, calling with both args and kwargs is allowed, in EXIR, only args
217-
will be used, kwargs will be an empty dict.
218-
3. In EXIR, constant arguments will be embedded within the graph.
216+
2. In EXIR, constant arguments will be embedded within the graph.
217+
3. In FX graph, a get_attr node can represent reading any attribute stored in
218+
the graph module. However, in EXIR this will only be reading submodules as all
219+
parameters/buffers will be passed in as inputs to the graph module.
219220

220221

221222
#### Metadata

0 commit comments

Comments
 (0)