Skip to content

Commit

Permalink
Remove CPU support for lamb (#1796)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: #1796

Pull Request resolved: #1772

This diff is a part of FBGEMM TBE optimizer deprecation plan (see
#1727)

Changes include:
- Setting `has_cpu_support` of `lamb` to
  `False` in the code gen script
- Updating `codegen/TARGETS` and `CMakeLists.txt`

See D45835048 for how the changes affect the code generation

Reviewed By: csmiler

Differential Revision: D45845113

fbshipit-source-id: fb35dc49d7301960961094a02342d4e35c614dbb
  • Loading branch information
q10 authored and facebook-github-bot committed Jun 1, 2023
1 parent 05bf018 commit 58ecab6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion fbgemm_gpu/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,6 @@ file(GLOB_RECURSE asmjit_sources

set(COMMON_OPTIMIZERS
adagrad
lamb
lars_sgd
partial_rowwise_adam
partial_rowwise_lamb
Expand All @@ -179,6 +178,7 @@ set(CPU_ONLY_OPTIMIZERS "")

set(GPU_ONLY_OPTIMIZERS
adam
lamb
rowwise_adagrad_with_weight_decay
approx_rowwise_adagrad_with_weight_decay)

Expand Down
2 changes: 1 addition & 1 deletion fbgemm_gpu/codegen/embedding_backward_code_generator.py
Original file line number Diff line number Diff line change
Expand Up @@ -1129,7 +1129,7 @@ def lamb() -> None:
split_weight_update=split_weight_update,
split_post_update="",
split_weight_update_cpu=split_weight_update_cpu,
has_cpu_support=True,
has_cpu_support=False,
has_gpu_support=True,
has_vbe_support=False,
)
Expand Down

0 comments on commit 58ecab6

Please sign in to comment.