Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44194 )
Change subject: (test)lz4/lib: use 'std=c17'
......................................................................
Patch Set 1: Code-Review-1
Patch Set 1: Code-Review-1
Patch Set 1: Code-Review-1
This code is imported verbatim from LZ4's GitHub, so let's not touch it unless necessary.
I've checked, at GitHub, they don't even use 'std='. So by default, GCC will use c17.
Also, it seems, that we are using an old version.
AFAIUI, GCC by default uses *gnu17* according to the documentation: https://gcc.gnu.org/onlinedocs/gcc/Standards.html
The default, if no C language dialect options are given, is -std=gnu17.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/44194
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ib7f464f31198155feb2a7c249cc2a0a8ab60e3ba
Gerrit-Change-Number: 44194
Gerrit-PatchSet: 1
Gerrit-Owner: HAOUAS Elyes
ehaouas@noos.fr
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: HAOUAS Elyes
ehaouas@noos.fr
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Thu, 06 Aug 2020 11:19:52 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment