-
Notifications
You must be signed in to change notification settings - Fork 249
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
fix(cpp1): contract check invalid alternative #491
Conversation
You can see the generated code executing at: https://cpp2.godbolt.org/z/8Kvdhc6aG. |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as duplicate.
This comment was marked as resolved.
This comment was marked as resolved.
Feel free to merge this until someone resolves #492 to move the logic to compile-time. |
This is the diff between # test-results]$
for f in `ls gcc-10/* gcc-13/* | sed 's/gcc-[[:digit:]]\+\///' | sort --unique`; do diff gcc-10/$f gcc-13/$f; done 1,2c1,2
< g++-10 (Ubuntu 10.3.0-1ubuntu1~20.04) 10.3.0
< Copyright (C) 2020 Free Software Foundation, Inc.
---
> c++ (GCC) 13.1.1 20230429
> Copyright (C) 2023 Free Software Foundation, Inc.
diff: gcc-10/pure2-bugfix-for-runtime-invalid-alternative.cpp.execution: No such file or directory
diff: gcc-10/pure2-bugfix-for-runtime-invalid-alternative.cpp.output: No such file or directory
diff: gcc-13/pure2-cpp1-multitoken-fundamental-types.cpp.execution: No such file or directory
diff: gcc-13/pure2-inspect-values.cpp.execution: No such file or directory
diff: gcc-13/run-tests-gcc-10.sh: No such file or directory
For convenience, I think a result of a newer compiler versions should only be in the repository if it is different from the previous compiler version. |
f6c844b
to
095edee
Compare
Marking as draft to prevent merging |
Resolves #433.
I have some simplifications in store.
They might come in further commits,
because according to #433 (comment),
they might not work.
Testing summary.
Acknowledgements.