Skip to content

[rtsan][NFC] Rename *interceptors.cpp to *interceptors_posix.cpp #112935

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

cjappl
Copy link
Contributor

@cjappl cjappl commented Oct 18, 2024

This is a straight rename, done in preparation of exploring rtsan on windows.

Please be aware of what is not here, I left this header unchanged, as I believe it should be expected that any system implementing rtsan should adhere to this API:

https://github.com/llvm/llvm-project/blob/main/compiler-rt/lib/rtsan/rtsan_interceptors.h

@cjappl
Copy link
Contributor Author

cjappl commented Oct 18, 2024

CC @jatinchowdhury18

@llvmbot
Copy link
Member

llvmbot commented Oct 18, 2024

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Chris Apple (cjappl)

Changes

This is a straight rename, done in preparation of exploring rtsan on windows.

Please be aware of what is not here, I left this header unchanged, as I believe it should be expected that any system implementing rtsan should adhere to this API:

https://github.com/llvm/llvm-project/blob/main/compiler-rt/lib/rtsan/rtsan_interceptors.h


Full diff: https://github.com/llvm/llvm-project/pull/112935.diff

4 Files Affected:

  • (modified) compiler-rt/lib/rtsan/CMakeLists.txt (+1-1)
  • (renamed) compiler-rt/lib/rtsan/rtsan_interceptors_posix.cpp ()
  • (modified) compiler-rt/lib/rtsan/tests/CMakeLists.txt (+1-1)
  • (renamed) compiler-rt/lib/rtsan/tests/rtsan_test_interceptors_posix.cpp ()
diff --git a/compiler-rt/lib/rtsan/CMakeLists.txt b/compiler-rt/lib/rtsan/CMakeLists.txt
index f8dd4d735bc2a3..a4413d9992b62a 100644
--- a/compiler-rt/lib/rtsan/CMakeLists.txt
+++ b/compiler-rt/lib/rtsan/CMakeLists.txt
@@ -5,7 +5,7 @@ set(RTSAN_CXX_SOURCES
   rtsan_context.cpp
   rtsan_diagnostics.cpp
   rtsan_flags.cpp
-  rtsan_interceptors.cpp
+  rtsan_interceptors_posix.cpp
   rtsan_stats.cpp
   rtsan_suppressions.cpp
   )
diff --git a/compiler-rt/lib/rtsan/rtsan_interceptors.cpp b/compiler-rt/lib/rtsan/rtsan_interceptors_posix.cpp
similarity index 100%
rename from compiler-rt/lib/rtsan/rtsan_interceptors.cpp
rename to compiler-rt/lib/rtsan/rtsan_interceptors_posix.cpp
diff --git a/compiler-rt/lib/rtsan/tests/CMakeLists.txt b/compiler-rt/lib/rtsan/tests/CMakeLists.txt
index 139eea785fcdca..0cf07b307d4610 100644
--- a/compiler-rt/lib/rtsan/tests/CMakeLists.txt
+++ b/compiler-rt/lib/rtsan/tests/CMakeLists.txt
@@ -16,7 +16,7 @@ set(RTSAN_UNITTEST_CFLAGS
 
 set(RTSAN_INST_TEST_SOURCES
     rtsan_test_functional.cpp
-    rtsan_test_interceptors.cpp
+    rtsan_test_interceptors_posix.cpp
     rtsan_test_main.cpp)
 
 set(RTSAN_NOINST_TEST_SOURCES
diff --git a/compiler-rt/lib/rtsan/tests/rtsan_test_interceptors.cpp b/compiler-rt/lib/rtsan/tests/rtsan_test_interceptors_posix.cpp
similarity index 100%
rename from compiler-rt/lib/rtsan/tests/rtsan_test_interceptors.cpp
rename to compiler-rt/lib/rtsan/tests/rtsan_test_interceptors_posix.cpp

@cjappl cjappl merged commit 1e07c48 into llvm:main Oct 21, 2024
10 checks passed
@cjappl cjappl deleted the rename_posix branch October 21, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants