You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by "gustavonihei (via GitHub)" <gi...@apache.org> on 2023/01/26 12:16:25 UTC

[GitHub] [nuttx] gustavonihei commented on a diff in pull request #8244: LLVM libcxx upgrade from 12.0.0 to 15.0.7

gustavonihei commented on code in PR #8244:
URL: https://github.com/apache/nuttx/pull/8244#discussion_r1087777629


##########
libs/libxx/Kconfig:
##########
@@ -87,7 +87,7 @@ endif
 
 config CXX_STANDARD
 	string "Language standard"
-	default "gnu++17"
+	default "gnu++20"

Review Comment:
   We shouldn't increase the default to `gnu++20`.
   Not even the latest version of GCC has full support for the Language features:
   https://gcc.gnu.org/projects/cxx-status.html#cxx20
   
   Even for C++17 is states:
   > GCC has almost full support for the previous revision of the C++ standard, which was published in 2017.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org