[llvm] 0184889 - [mlgo] Don't set the source permissions when copying over tf files

Mircea Trofin via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 26 09:51:01 PDT 2022


Author: Mircea Trofin
Date: 2022-08-26T09:49:53-07:00
New Revision: 0184889e0dd5e77954c772e233860cbbcb722ac4

URL: https://github.com/llvm/llvm-project/commit/0184889e0dd5e77954c772e233860cbbcb722ac4
DIFF: https://github.com/llvm/llvm-project/commit/0184889e0dd5e77954c772e233860cbbcb722ac4.diff

LOG: [mlgo] Don't set the source permissions when copying over tf files

This is akin to the `install` behavior. It should make build directory
deletable on certain build bots.

Added: 
    

Modified: 
    llvm/CMakeLists.txt

Removed: 
    


################################################################################
diff  --git a/llvm/CMakeLists.txt b/llvm/CMakeLists.txt
index cdce25bec844..8a1dcb39a147 100644
--- a/llvm/CMakeLists.txt
+++ b/llvm/CMakeLists.txt
@@ -994,6 +994,7 @@ if (NOT TENSORFLOW_AOT_PATH STREQUAL "")
   # This needs to happen to avoid clashing protobuf codegen when building both AOT and development mode.
   # We plan to replace protobuf with a simpler alternative, so this will go away.
   file(COPY ${TENSORFLOW_AOT_PATH}/include DESTINATION ${CMAKE_LIBRARY_OUTPUT_DIRECTORY}/tensorflow
+    NO_SOURCE_PERMISSIONS
     PATTERN "google/*" EXCLUDE
     PATTERN "*.pb.h" EXCLUDE) 
   include_directories(${CMAKE_LIBRARY_OUTPUT_DIRECTORY}/tensorflow/include)


        


More information about the llvm-commits mailing list