Add stdint.h include to emitterutils.cpp#1450
Closed
taotieren wants to merge 1 commit into
Closed
Conversation
Collaborator
|
Thank you for your contribution. cstdint is the correct c++ header for uint16_t. stdint.h is a c header and not required. So I am assuming the issue must lie somwhere else. Could you report your gcc/clang version and which OS you are using? |
Author
❯ lsb_release -a
LSB Version: n/a
Distributor ID: Arch
Description: Arch Linux
Release: rolling
Codename: n/a
❯ uname -a
Linux 14s 7.0.12-zen1-1-zen #1 ZEN SMP PREEMPT_DYNAMIC Wed, 10 Jun 2026 08:57:50 +0000 x86_64 GNU/Linux
❯ gcc -v
使用内建 specs。
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-pc-linux-gnu/16.1.1/lto-wrapper
目标:x86_64-pc-linux-gnu
配置为:/build/gcc/src/gcc/configure --enable-languages=ada,c,c++,d,fortran,go,lto,m2,objc,obj-c++,rust,cobol --enable-bootstrap --prefix=/usr --libdir=/usr/lib --libexecdir=/usr/lib --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=https://gitlab.archlinux.org/archlinux/packaging/packages/gcc/-/issues --with-build-config=bootstrap-lto --with-linker-hash-style=gnu --with-system-zlib --enable-cet=auto --enable-checking=release --enable-clocale=gnu --enable-default-pie --enable-default-ssp --enable-gnu-indirect-function --enable-gnu-unique-object --enable-libstdcxx-backtrace --enable-link-serialization=1 --enable-linker-build-id --enable-lto --enable-multilib --enable-plugin --enable-shared --enable-threads=posix --disable-libssp --disable-libstdcxx-pch --disable-werror --disable-fixincludes
线程模型:posix
支持的 LTO 压缩算法:zlib zstd
gcc 版本 16.1.1 20260430 (GCC) After manually adding the stdint.h header file, the code compiles successfully. |
Collaborator
|
I believe you are using 0.8.0 or older and not the newest version 0.9.0 (or master branch). This issue has been fixed: |
Author
Yes. I just checked the version of yaml-cpp used by the upstream repo and it needs to be upgraded to 0.9.0 or above. Thanks a lot. |
Collaborator
|
Thank you for forwarding it to qsoc ! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.