[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria added a comment. Changed to the if condition. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155857/new/ https://reviews.llvm.org/D155857 ___ cfe-commits mailing list cfe-commits@lists.llvm.org h

[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria updated this revision to Diff 543244. fzakaria added a comment. The previous CMake variable wasn't working. Changed it to the pattern I see in the codebase. Confirmed it via `ninja` command and grepping. ❯ rg --color=always no-nonnull build.ninja | head -n 1 FLAGS = -fPIC -fno-sem

[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria added a comment. Ugh you are right. So weird -- This works: if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU") append("-Wno-nonnull" CMAKE_CXX_FLAGS) endif() I read the CMake documentation and it looked like the variable I used should have worked. https://cmake.org/cmake/help/latest/va

[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria marked an inline comment as done. fzakaria added a comment. @MaskRay thanks for the review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155857/new/ https://reviews.llvm.org/D155857 ___ cfe-com

[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria updated this revision to Diff 543238. fzakaria added a comment. Address feedback from maskray@ - imperative comment - switch to CMAKE_GNU_COMPILER_ID for variable Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155857/new/ https://reviews.l

[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria added a comment. @rsmith -- sounds good. Please commit on my behalf when you are ready as I don't have commit access. Thank you for the review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155857/new/ https://reviews.llvm.org/D155857 __

[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria updated this revision to Diff 542663. fzakaria added a comment. Added comment to explain why we are disabling Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155857/new/ https://reviews.llvm.org/D155857 Files: llvm/cmake/modules/HandleLLV

[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria added a comment. @rsmith and @MaskRay thank you for the feedback. I think this is more in line with what we discussed on the #LLVM chat Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155857/new/ https:/

[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria updated this revision to Diff 542614. fzakaria added a comment. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Updated instead to disable it in CMake Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D155857/new/ https:/

[PATCH] D155857: [clang] fix nonnull warnings during build

fzakaria created this revision. Herald added a project: All. fzakaria requested review of this revision. Herald added subscribers: cfe-commits, wangpc. Herald added a project: clang. I noticed during the build that GCC would emit a ton of nonnull warnings. Example: /usr/local/google/home/fmza