OSSForks/pytorch
1
0
Fork 0
mirror of https://github.com/zebrajr/pytorch.git synced 2025-12-06 12:20:52 +01:00
Code Issues Actions 38 Packages Projects Releases Wiki Activity

[BE] Build PyTorch with -Wnewline-eof (#99687)

Browse Source 
This would avoid further regressions like the ones reported in https://github.com/pytorch/pytorch/pull/96668#issuecomment-1468029259

Surround some ONNX/flatbuffer includes with `C10_DIAGNOSTIC_PUSH_AND_IGNORED_IF_DEFINED("-Wnewline-eof")` cone of shame

Fixes https://github.com/pytorch/pytorch/issues/96747

Pull Request resolved: https://github.com/pytorch/pytorch/pull/99687
Approved by: https://github.com/kit1980
This commit is contained in:
Nikita Shulga 2023-04-21 14:46:47 +00:00 committed by PyTorch MergeBot
parent dbf0db958f
commit 6b8ef8ea4c
7 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CMakeLists.txt
View File

@ -809,6 +809,7 @@ if(NOT MSVC)
append_cxx_flag_if_supported("-Wno-strict-overflow" CMAKE_CXX_FLAGS)
append_cxx_flag_if_supported("-Wno-strict-aliasing" CMAKE_CXX_FLAGS)
append_cxx_flag_if_supported("-Wvla-extension" CMAKE_CXX_FLAGS)
append_cxx_flag_if_supported("-Wnewline-eof" CMAKE_CXX_FLAGS)
if("${CMAKE_CXX_COMPILER_ID}" MATCHES "Clang")
string(APPEND CMAKE_CXX_FLAGS " -Wno-range-loop-analysis")
string(APPEND CMAKE_CXX_FLAGS " -Wno-pass-failed")

2
aten/src/ATen/native/transformers/cuda/flash_attn/fmha_utils.h
View File

@ -97,4 +97,4 @@ static inline size_t get_size_in_bytes( size_t n, Data_type dtype ) {
}
}
////////////////////////////////////////////////////////////////////////////////////////////////////
////////////////////////////////////////////////////////////////////////////////////////////////////

2
aten/src/ATen/native/transformers/sdp_utils_cpp.h
View File

@ -23,4 +23,4 @@ inline c10::SymFloat calculate_scale(
return c10::SymFloat(softmax_scale);
}
} // namespace sdp
} // namespace sdp

2
torch/csrc/jit/passes/onnx/constant_map.h
View File

@ -3,8 +3,10 @@
#include <c10/macros/Macros.h>
C10_DIAGNOSTIC_PUSH_AND_IGNORED_IF_DEFINED("-Wsuggest-override")
C10_DIAGNOSTIC_PUSH_AND_IGNORED_IF_DEFINED("-Wnewline-eof")
#include <onnx/shape_inference/implementation.h>
C10_DIAGNOSTIC_POP()
C10_DIAGNOSTIC_POP()
#include <torch/csrc/jit/ir/ir.h>
#include <torch/csrc/jit/serialization/export.h>

2
torch/csrc/jit/serialization/export.cpp
View File

@ -21,7 +21,9 @@
#include <torch/version.h>
#include <atomic>
C10_DIAGNOSTIC_PUSH_AND_IGNORED_IF_DEFINED("-Wnewline-eof")
#include <onnx/checker.h>
C10_DIAGNOSTIC_POP()
#include <onnx/onnx_pb.h>
#include <onnx/proto_utils.h>
C10_DIAGNOSTIC_PUSH_AND_IGNORED_IF_DEFINED("-Wsuggest-override")

2
torch/csrc/jit/serialization/mobile_bytecode_generated.h
View File

@ -4,7 +4,9 @@
#ifndef FLATBUFFERS_GENERATED_MOBILEBYTECODE_TORCH_JIT_MOBILE_SERIALIZATION_H_
#define FLATBUFFERS_GENERATED_MOBILEBYTECODE_TORCH_JIT_MOBILE_SERIALIZATION_H_
C10_DIAGNOSTIC_PUSH_AND_IGNORED_IF_DEFINED("-Wnewline-eof")
#include "flatbuffers/flatbuffers.h"
C10_DIAGNOSTIC_POP()
namespace torch {
namespace jit {

2
torch/csrc/utils/pythoncapi_compat.h
View File

@ -572,4 +572,4 @@ PyCode_GetCellvars(PyCodeObject *code)
#ifdef __cplusplus
}
#endif
#endif // PYTHONCAPI_COMPAT
#endif // PYTHONCAPI_COMPAT