Skip to content

Commit

Permalink
Mention the bug of Generator to suggest higher compiler bugs
Browse files Browse the repository at this point in the history
See #372
  • Loading branch information
ChuanqiXu9 committed Mar 12, 2024
1 parent fd350d7 commit ce035dc
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ Required Compiler: clang (>= 10.0.0) or gcc (>= 10.3) or Apple-clang (>= 14)

Note that we need to add the `-Wno-maybe-uninitialized` option when we use gcc 12 due to a false positive diagnostic message by gcc 12

Note that when using clang 15 it may be necessary to add the `-Wno-unsequenced` option, which is a false positive of clang 15. See https://github.com/llvm/llvm-project/issues/56768 for details.
Note that when using clang 15 it may be necessary to add the `-Wno-unsequenced` option, which is a false positive of clang 15. See https://github.com/llvm/llvm-project/issues/56768 for details. If you're using `async_simple::Generator`, there may be some compiler bugs in clang15. We suggest to use clang17 or higher for that.

If you meet any problem about MSVC Compiler Error C4737. Try to add the /EHa option to fix the problem.

Expand Down
2 changes: 1 addition & 1 deletion README_CN.md
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ async\_simple 涉及 C++20 协程,对编译器版本有较高要求。需要 c

注意当使用 gcc12 时需要加上 `-Wno-maybe-uninitialized` 选项因为 gcc12 的一个误报。详情可见 https://github.com/alibaba/async_simple/issues/234。

注意当使用 clang15 时可能需要加上 `-Wno-unsequenced` 选项,这是clang15的一个误报。详情可见 https://github.com/llvm/llvm-project/issues/56768。
注意当使用 clang15 时可能需要加上 `-Wno-unsequenced` 选项,这是clang15的一个误报。详情可见 https://github.com/llvm/llvm-project/issues/56768。若你需要使用 `async_simple::Generator`,我们推荐使用 clang17 或更高的版本因为在低版本 clang 上存在一些关于 Generator 的 bug report。

注意,如果使用msvc时遇到了C4737错误,请加上选项/EHa。

Expand Down

0 comments on commit ce035dc

Please sign in to comment.