main

文件	最后提交记录	最后更新时间
CMakeLists.txt	Update cmake files with Windows related changes (#5369) The changes mainly affect compilation options. Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>	1 年前
GPUHello.cpp	Audit file-private C++ functions to ensure correct linkage (#6237) ### Summary I’ve gone through the C++ sources and updated functions that were file-local but not declared static or placed in anonymous namespaces. These can lead to symbol visibility issues when linking multiple translation units together. This patch: * Marks helper functions in bin/triton-tensor-layout.cpp as static. * Marks internal variables and helpers in lib/Instrumentation/PrintLoadStoreMemSpaces.cpp as static. * Adds static to a number of internal free functions and templates in third_party/f2reduce/f2reduce.cpp. * Corrects linkage for a handful of AMD GPU transforms and stream utilities that were file-local. * Makes file-local helpers in third_party/nvidia/lib/NVGPUToLLVM/NVGPUToLLVMPass.cpp static. * Ensures initProton in third_party/proton/csrc/Proton.cpp is static. * Tidies up similar file-local helpers in test code. I initially made some functions static that were already declared in headers; that’s been reverted so things compile cleanly. All C++ unit tests and lit tests pass, and pre-commit run --from-ref origin/main --to-ref HEAD shows no outstanding issues. ### New contributor declaration - [x] I am not making a trivial change, such as fixing a typo in a comment. - [x] I have written a PR description following these [rules](https://cbea.ms/git-commit/#why-not-how). - [x] I have run pre-commit run --from-ref origin/main --to-ref HEAD. - [x] This PR does not need a test because this is a refactor that only adjusts linkage of file-local functions. - [x] I have not added any lit tests. All tests (make test-nogpu) pass: Testing Time: 1.07s Total Discovered Tests: 131 Passed: 131 (100.00%) 100% tests passed, 0 tests failed out of 207 Total Test time (real) = 1.48 sec	1 年前