Branch: refs/heads/main
Revision: 2c1aefa
Author: aconsroe-hpe
Link: Add missing braces for compiler build error by aconsroe-hpe · Pull Request #18931 · chapel-lang/chapel · GitHub
Log Message:
Merge pull request #18931 from aconsroe-hpe/fix/missing-braces
Add missing braces for compiler build error
reviewed by @ronawho
std::array needed another pair of braces when -Wmissing-braces was
on. When assigning with = you can drop the outermost one, but not here
Adds -Wmissing-braces to the compiler CXX flags so
we (I!) don't hit this in the future. Only added to gnu and clang as
those will be used in development (and these are warnings enabled with
CHPL_DEVELOPER)
Fixes a build error reported by the gasnet team. Thanks!
Note that it seems clang doesn't report this warning, but I've still
added it to Makefile.clang
Modified Files:
M compiler/passes/buildDefaultFunctions.cpp
M make/compiler/Makefile.clang
M make/compiler/Makefile.gnu
Compare: https://github.com/chapel-lang/chapel/compare/5a84617dae6e...2c1aefa56ba1