There seems to be no evidence that having internal linkage for the function
was intentional. Since 'static' functions are normally used only in .cpp
files, using 'inline' in the header file is more appropriate.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Build result: fail - 33519 tests passed, 1 failed and 464 were skipped.
failed: LLVM.tools/llvm-ar/mri-utf8.test
Log files: cmake-log.txt, ninja_check_all-log.txt, CMakeCache.txt