diff options
| author | Dimitry Andric <dimitry@andric.com> | 2017-12-24 21:11:32 +0000 |
|---|---|---|
| committer | Dimitry Andric <dimitry@andric.com> | 2017-12-24 21:11:32 +0000 |
| commit | 60c7f2d9ce54552e53c252d9f347ffd5223d3695 (patch) | |
| tree | b43ee7a72bf1ba43731b73d9e7f31c7a3f605a59 /compiler-rt/lib/builtins | |
| parent | a9c490b71dbf1bc3f657527e2a268448529cdfd4 (diff) | |
| download | bcm5719-llvm-60c7f2d9ce54552e53c252d9f347ffd5223d3695.tar.gz bcm5719-llvm-60c7f2d9ce54552e53c252d9f347ffd5223d3695.zip | |
Fix PR35739: chkstk and chkst2 should only be built for Windows
As reported in PR35739, rL252927 added the Windows specific chkstk and
chkstk2 sources unconditionally, and since these are assembly without a
NO_EXEC_STACK_DIRECTIVE at the end, automated vulnerability scanners
warned about the objects having an executable stack.
Avoid the problem by only including these files when Windows is
targeted.
Reviewers: compnerd, rnk, martell
Reviewed By: martell
Subscribers: mstorsjo, mgorny, martell, javed.absar, #sanitizers, llvm-commits
Differential Revision: https://reviews.llvm.org/D41567
llvm-svn: 321431
Diffstat (limited to 'compiler-rt/lib/builtins')
| -rw-r--r-- | compiler-rt/lib/builtins/CMakeLists.txt | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/compiler-rt/lib/builtins/CMakeLists.txt b/compiler-rt/lib/builtins/CMakeLists.txt index 6a0faf8ebf0..0b50b5bb8d2 100644 --- a/compiler-rt/lib/builtins/CMakeLists.txt +++ b/compiler-rt/lib/builtins/CMakeLists.txt @@ -232,8 +232,6 @@ set(x86_ARCH_SOURCES if (NOT MSVC) set(x86_64_SOURCES - x86_64/chkstk.S - x86_64/chkstk2.S x86_64/floatdidf.c x86_64/floatdisf.c x86_64/floatdixf.c @@ -253,8 +251,6 @@ if (NOT MSVC) set(i386_SOURCES i386/ashldi3.S i386/ashrdi3.S - i386/chkstk.S - i386/chkstk2.S i386/divdi3.S i386/floatdidf.S i386/floatdisf.S |
