Skip to content

Raise error when autotune_with_torch_compile_fusion is set without torch_compile_fusion#1951

Open
yf225 wants to merge 2 commits intomainfrom
willfeng/error-on-fusion-autotune-without-fusion
Open

Raise error when autotune_with_torch_compile_fusion is set without torch_compile_fusion#1951
yf225 wants to merge 2 commits intomainfrom
willfeng/error-on-fusion-autotune-without-fusion

Conversation

@yf225
Copy link
Copy Markdown
Contributor

@yf225 yf225 commented Apr 4, 2026

Summary

  • Previously, setting autotune_with_torch_compile_fusion=True without torch_compile_fusion=True was silently ignored.
  • Now it raises FusionAutotuneRequiresTorchCompileFusion at Settings construction time with a clear error message.

Test plan

  • Verify Settings(autotune_with_torch_compile_fusion=True) raises the error
  • Verify Settings(autotune_with_torch_compile_fusion=True, torch_compile_fusion=True) works fine
  • Existing tests pass

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Apr 4, 2026
@yf225 yf225 requested review from jansel, oulgen and shunting314 April 4, 2026 23:54
…rch_compile_fusion

Previously, setting autotune_with_torch_compile_fusion=True without
torch_compile_fusion=True was silently ignored. Now it raises
FusionAutotuneRequiresTorchCompileFusion at Settings construction time.
@yf225 yf225 force-pushed the willfeng/error-on-fusion-autotune-without-fusion branch from 30f0c1c to 37da98b Compare April 4, 2026 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant