Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zq/add specified autocompare2 #794

Merged

Conversation

NeosZhang
Copy link
Collaborator

@NeosZhang NeosZhang commented Apr 25, 2024

基于这个PR进行修改,原来的PR保留为draft。
重构register机制,针对配置文件中的custom_fallback和autocmpare,定义了四个注册宏
生成的代码片段如下:
image

@@ -2770,6 +2768,7 @@

- schema: _amp_foreach_non_finite_check_and_unscale_(at::TensorList self, Tensor(b!) found_inf, Tensor inv_scale) -> void
autocompare: disable
# TODO(someone): fix this issue when `autocompare` is on
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

谁记得这件事就写谁

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done,后面我来跟进这个issue

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已经修复了

[&opname](auto& matcher) { return std::regex_match(opname, matcher); });
}

constexpr const char* fallbackEnvName = "DIPU_FORCE_FALLBACK_OPS_LIST";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
constexpr const char* fallbackEnvName = "DIPU_FORCE_FALLBACK_OPS_LIST";
constexpr const char* kFallbackEnvName = "DIPU_FORCE_FALLBACK_OPS_LIST";

下同

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

还有俩

@NeosZhang NeosZhang requested a review from lljbash April 26, 2024 09:04
@mrdanielw mrdanielw merged commit 2200540 into DeepLink-org:main Apr 28, 2024
30 checks passed
@mrdanielw mrdanielw deleted the zq/add_specified_autocompare2 branch April 28, 2024 04:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants