This is an archive of the discontinued LLVM Phabricator instance.

Differential D146472

[libc][obvious] fix missing type headers
ClosedPublic

Authored by michaelrj on Mar 20 2023, 4:42 PM.

Details

Reviewers
None
Commits
rG07cdf4a10d78: [libc][obvious] fix missing type headers
Summary

forgot to add wint_t and wchar_t type headers to the wchar
headergen definition.

Diff Detail

Event Timeline

michaelrj created this revision.Mar 20 2023, 4:42 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMar 20 2023, 4:42 PM
Herald added subscribers: libc-commits, mikhail.ramalho, ecnelises, tschuett. · View Herald Transcript
michaelrj requested review of this revision.Mar 20 2023, 4:42 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 20 2023, 4:43 PM
This revision was landed with ongoing or failed builds.
Closed by commit rG07cdf4a10d78: [libc][obvious] fix missing type headers (authored by michaelrj). · Explain Why
This revision was automatically updated to reflect the committed changes.
michaelrj added a commit: rG07cdf4a10d78: [libc][obvious] fix missing type headers.
Harbormaster completed remote builds in B220580: Diff 506774.Mar 20 2023, 4:48 PM

Revision Contents

PathSize
libc/
include/
3 lines

Diff 506776

libc/include/CMakeLists.txt

Show First 20 Lines • Show All 469 Lines • ▼ Show 20 Lines
add_gen_header( add_gen_header(
wchar wchar
DEF_FILE wchar.h.def DEF_FILE wchar.h.def
GEN_HDR wchar.h GEN_HDR wchar.h
DEPENDS DEPENDS
.llvm_libc_common_h .llvm_libc_common_h
.llvm-libc-macros.wchar_macros .llvm-libc-macros.wchar_macros
.llvm-libc-types.size_t
.llvm-libc-types.wint_t
.llvm-libc-types.wchar_t
) )
if(NOT LLVM_LIBC_FULL_BUILD) if(NOT LLVM_LIBC_FULL_BUILD)
# We don't install headers in non-fullbuild mode. # We don't install headers in non-fullbuild mode.
return() return()
endif() endif()
function(get_all_install_header_targets out_var) function(get_all_install_header_targets out_var)
Show All 29 Lines