Skip to content
Success

Changes

Summary

  1. Added a check in operatorLookup so bad operators give a proper error message instead of segment fault. (details)
  2. remove superflous calls to std::move (details)
  3. Attempted to inject some examples (and some other pieces I had missed) into the enum proposal. (details)
  4. fix length for juxtaposed strings: "ABC" "DEF" (details)
  5. add another attribute test (details)
  6. update expected for attribute test aarch64 (details)
  7. update expected for attribute test x86 (details)
  8. more proofreading of introduction chapter (details)
  9. Removed increment and decrement operators for enumerations from builtins. I am confident the generic fallbacks work because I added tests. (details)
  10. Explicate the present state for test-suite programs being free of warnings. (details)
  11. Remove unnecessary special case for zero_t in code gen that caused unused-variable warning in thunks. (details)
  12. remove warnings from alloc_internal$ in stdlib.hfa (details)
  13. put back parameter names in comments (details)
  14. remove fixed tests from WFLGAS_OPT_LAX (details)
  15. change wflags-orig.txt to wflags-lax.txt (details)
  16. Solved the requested warning with exceptions. Also went through the exceptions tests and managed to remove about 2/3rds of them from the lax list that now either didn't have any warnings or warnings because of the test itself. (details)
  17. Try to get full build completing by disabling a data-collection substep. (details)
  18. Fix test-classification commands to detect more strict-capable tests.  Remove such tests from the lax list. (details)
  19. Add file missed in d9162ec9 (details)
  20. Add completion of work in progress mistakenly included in 584612d.  Fix test string-api-coverage to be warning free. (details)
  21. Changed some inline declarations to static or static inline (which are the same except for intent communication). This makes them more robust to inline changes. (details)
  22. fix build problem by fixing unsupported C23 attribute in test (details)
  23. update architecture expect file for ARM (details)
  24. update architecture expect file for x86 (details)
  25. update architecture expect file for x64 (details)
  26. Add back overnight build's benchmark compile attribute data point (details)
  27. Remove warnings due to unused parameters in generated code for zero-length ttype instantiations. (details)
  28. Remove test-code-induced warnings from Array RAII. (details)
  29. Try to fix full build by suppressing known warning's -W code under gcc 7-8-9. (details)
Commit 15cb7900f3b1c8c8879fb98afd3fbaa2455f3f82 by ajbeach
Added a check in operatorLookup so bad operators give a proper error message instead of segment fault.
The file was modifiedsrc/CodeGen/OperatorTable.cpp
Commit 4167afa8457c6663f553f8e13b8cff5feb258535 by Peter Buhr
remove superflous calls to std::move
The file was modifiedsrc/Validate/ReplaceTypedef.cpp
Commit 80018f56b169cc00fa49f0bff2f4f084646b61bb by ajbeach
Attempted to inject some examples (and some other pieces I had missed) into the enum proposal.
The file was modifieddoc/proposals/enums.md
Commit f9a0dd0ededb566dea1f3d1f79526bed77c8063d by Peter Buhr
fix length for juxtaposed strings: "ABC" "DEF"
The file was modifiedsrc/Parser/ExpressionNode.cpp
Commit 9a01745e06266b363221bee8eedb6c563702cf33 by Peter Buhr
add another attribute test
The file was modifiedtests/attributes.cfa
The file was modifiedtests/.expect/attributes.x64.txt
Commit 9b55aa3a1e4047e501ad55d95cfc61aab206ce42 by Peter Buhr
update expected for attribute test aarch64
The file was modifiedtests/.expect/attributes.arm64.txt
Commit 4ff7ea399fa9e13495f9b87399cfaa90956f0091 by Peter Buhr
update expected for attribute test x86
The file was modifiedtests/.expect/attributes.x86.txt
Commit 2980ccb81bba22ad3dcfcfd13e3857a6f2909ee3 by Peter Buhr
more proofreading of introduction chapter
The file was modifieddoc/theses/fangren_yu_MMath/intro.tex
Commit 5780d0daa4264756677240ebbf2a1ad74597dadb by ajbeach
Removed increment and decrement operators for enumerations from builtins. I am confident the generic fallbacks work because I added tests.
The file was modifiedtests/.expect/declarationSpecifier.x64.txt
The file was modifiedtests/.expect/gccExtensions.x64.txt
The file was modifiedtests/.expect/declarationSpecifier.arm64.txt
The file was modifiedtests/.expect/gccExtensions.x86.txt
The file was modifiedtests/.expect/extension.x64.txt
The file was modifiedtests/.expect/extension.arm64.txt
The file was modifiedtests/.expect/gccExtensions.arm64.txt
The file was modifiedtests/.expect/KRfunctions.arm64.txt
The file was modifiedtests/.expect/extension.x86.txt
The file was addedtests/enum_tests/inc-dec.cfa
The file was modifiedtests/.expect/declarationSpecifier.x86.txt
The file was modifiedtests/.expect/KRfunctions.x64.txt
The file was addedtests/enum_tests/.expect/inc-dec.txt
The file was modifiedtests/.expect/KRfunctions.x86.txt
The file was modifiedlibcfa/prelude/builtins.c
Commit faf9e1980e17d288281279a8ee128978108933dd by Michael Brooks
Explicate the present state for test-suite programs being free of warnings.

Now, test-suite compilation occurs in either strict or lax "wflags" mode.  Strict mode is equivalent to what used to be done for the `nowarn` folder, lax to the rest of the test suite.  Makefile.am now owns a list of tests that opt for lax wflags.  About 70% of the tests opt for lax.

Remove special handling for the `nowarn` folder.  Now, it simply does not appear on the opt-for-lax list.

Remove former tests
nowarn/fstream
nowarn/list
nowarn/string
because, respectively
io/io
list/dlist-insert-remove
collections/string-api-coverage
obviate them by getting strict wflags.
The file was removedtests/nowarn/.expect/fstream.txt
The file was removedtests/nowarn/.expect/string.txt
The file was removedtests/nowarn/fstream.cfa
The file was removedtests/nowarn/string.cfa
The file was modifiedtests/Makefile.am
The file was removedtests/nowarn/.expect/list.txt
The file was removedtests/nowarn/list.cfa
Commit e0330d2cd1a09857e7eda48e70140f0f25161679 by Michael Brooks
Remove unnecessary special case for zero_t in code gen that caused unused-variable warning in thunks.

Add instructions for tidying warnings (with the present zero_t case as its running example) as a "proposal."

The code gen change is safe because a corresponding case for one_t was always "missing" and the the variable at issue is never used meaningfully.  (Prior work has size_t lowered as int.  While that's a dubious choice, it's beyond what I'm fixing here.)

Mark these tests as now clean of warnings:
  array-collections/array-basic
  ctrl-flow/loopctrl
  vector
  vector_math/vec2_int
  vector_math/vec2_float
  vector_math/vec3_float
  vector_math/vec4_float
  zero_one
The file was addeddoc/proposals/nowarn.md
The file was modifiedsrc/CodeGen/CodeGenerator.cpp
The file was modifiedtests/Makefile.am
The file was modifiedtests/array-collections/array-basic.cfa
The file was addedtests/nowarn/.expect/zero-thunk.txt
The file was addedtests/nowarn/zero-thunk.cfa
Commit e0f3bd23f623d604f6d18aa73e89c0388f37e424 by Peter Buhr
remove warnings from alloc_internal$ in stdlib.hfa
The file was modifiedlibcfa/src/stdlib.hfa
Commit a65cd5e945fecac9d8b11730d6bc465aac99fb4a by Peter Buhr
put back parameter names in comments
The file was modifiedlibcfa/src/stdlib.hfa
Commit 956ad355a0628c1f890198e8dbe8aa3babcff706 by Peter Buhr
remove fixed tests from WFLGAS_OPT_LAX
The file was modifiedtests/Makefile.am
Commit df91e154efd3628ba991d611e95933a2583500c3 by Peter Buhr
change wflags-orig.txt to wflags-lax.txt
The file was modifieddoc/proposals/nowarn.md
Commit d3cf623098e72d98fd3ea76544061496946dbba4 by ajbeach
Solved the requested warning with exceptions. Also went through the exceptions tests and managed to remove about 2/3rds of them from the lax list that now either didn't have any warnings or warnings because of the test itself.
The file was modifiedtests/Makefile.am
The file was modifiedtests/exceptions/cancel/thread.cfa
The file was modifiedtests/exceptions/finally.cfa
The file was modifiedsrc/Virtual/Tables.cpp
The file was modifiedtests/exceptions/conditional.cfa
The file was modifiedtests/exceptions/defaults.cfa
The file was modifiedtests/exceptions/resume.cfa
The file was modifiedtests/exceptions/try-ctrl-flow.cfa
The file was modifiedtests/exceptions/cancel/coroutine.cfa
The file was modifiedtests/exceptions/try-leave-catch.cfa
The file was modifiedtests/exceptions/virtual-cast.cfa
The file was modifiedtests/exceptions/data-except.cfa
The file was modifiedtests/exceptions/interact.cfa
The file was modifiedtests/exceptions/polymorphic.cfa
The file was modifiedtests/exceptions/terminate.cfa
The file was modifiedtests/exceptions/trash.cfa
Commit b05d79dfa93934939aadd307627a2beecfefb986 by Michael Brooks
Try to get full build completing by disabling a data-collection substep.

In benchmarks/compile.csv, column 'attributes' is now showing hardcoded zero.

Also, add log comment to ease similar troubleshooting in the future.
The file was modifiedbenchmark/Makefile.am
Commit d9162ec9cf76ec3a23094dfca0cd12499ddaf8e9 by Michael Brooks
Fix test-classification commands to detect more strict-capable tests.  Remove such tests from the lax list.

Nothing about how to use the document is different, just what to paste.

Also, add back tests that give warnings only on release mode.
The file was modifieddoc/proposals/nowarn.md
Commit 584612d49519a520b9b9f6c479417deec445ec6a by Michael Brooks
Add file missed in d9162ec9
The file was modifiedtests/Makefile.am
Commit b12b1baa9706d4fd9b5c66a157ab809d9cdf268a by Michael Brooks
Add completion of work in progress mistakenly included in 584612d.  Fix test string-api-coverage to be warning free.
The file was modifiedtests/collections/string-api-coverage.cfa
Commit 5251c6b3e179b007a0d6e799699a2ace9204dda2 by ajbeach
Changed some inline declarations to static or static inline (which are the same except for intent communication). This makes them more robust to inline changes.
The file was modifiedlibcfa/src/bits/stack.hfa
The file was modifiedtests/exceptions/except-io.hfa
The file was modifiedlibcfa/src/concurrency/coroutine.cfa
Commit 77148b08d37e54e1e2289ee68f8928d6326cb3bd by Peter Buhr
fix build problem by fixing unsupported C23 attribute in test
The file was modifiedtests/attributes.cfa
Commit 9dc05782e60cfd80ba650343f1f4309cb1ec3016 by Peter Buhr
update architecture expect file for ARM
The file was modifiedtests/.expect/attributes.arm64.txt
Commit d1566d4ab9946328dc76fdeec90ca9ff016dd45a by Peter Buhr
update architecture expect file for x86
The file was modifiedtests/.expect/attributes.x86.txt
Commit 50cad32590a9714a0cbf3e31088adf8411ea10a6 by Peter Buhr
update architecture expect file for x64
The file was modifiedtests/.expect/attributes.x64.txt
Commit 3df3a8f4dcb3e0d460f00091173a9d5682068922 by Michael Brooks
Add back overnight build's benchmark compile attribute data point
The file was modifiedbenchmark/Makefile.am
Commit 0bf03ba22468e758a0594078343f9671379485c3 by Michael Brooks
Remove warnings due to unused parameters in generated code for zero-length ttype instantiations.
The file was modifiedsrc/GenPoly/Specialize.cpp
The file was modifiedsrc/InitTweak/FixInit.cpp
The file was modifiedtests/Makefile.am
The file was modifiedtests/nowarn/unused.cfa
Commit 0cd168feb19ad0f03ac91f3787a06e392f60a55e by Michael Brooks
Remove test-code-induced warnings from Array RAII.

(Depends on zero-length tuples' fix of 0bf03ba224.)
The file was modifiedtests/Makefile.am
The file was modifiedtests/array-collections/array-raii.hfa
Commit 28c2c9d5402ae8f3d59f6810a9f090bb832eaad8 by Michael Brooks
Try to fix full build by suppressing known warning's -W code under gcc 7-8-9.
The file was modifiedtests/array-collections/array-raii.hfa