Skip to content

Update compiler-rt/include/profile from LLVM 20.1.4 to 21.1.8#26152

Merged
aheejin merged 1 commit intoemscripten-core:mainfrom
aheejin:update_compiler_rt_21_include_profile
Jan 23, 2026
Merged

Update compiler-rt/include/profile from LLVM 20.1.4 to 21.1.8#26152
aheejin merged 1 commit intoemscripten-core:mainfrom
aheejin:update_compiler_rt_21_include_profile

Conversation

@aheejin
Copy link
Member

@aheejin aheejin commented Jan 23, 2026

include/profile and lib/profile have been added to compiler-rt since #24160, but only lib/profile was added to update_compiler_rt.py, which caused this directory to be missing from the subsequent LLVM library updates.

`include/profile` and `lib/profile` have been added to compiler-rt in
 emscripten-core#24160, but only `lib/profile` has been added to
`update_compiler_rt.py`, which caused this directory missing from the
subsequent LLVM library updates.
@aheejin aheejin requested a review from sbc100 January 23, 2026 01:13
} // namespace __asan

#endif // !SANITIZER_FUCHSIA && !SANITIZER_RTEMS && !SANITIZER_EMSCRIPTEN
#endif // !SANITIZER_FUCHSIA && !SANITIZER_EMSCRIPTEN
Copy link
Member Author

@aheejin aheejin Jan 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drive-by fix: SANITIZER_RTEMS doesn't seem to exist anymore (It's also gone from the upstream)

@aheejin aheejin merged commit dc2aa93 into emscripten-core:main Jan 23, 2026
36 checks passed
@aheejin aheejin deleted the update_compiler_rt_21_include_profile branch January 23, 2026 05:09
inolen pushed a commit to inolen/emscripten that referenced this pull request Feb 13, 2026
…pten-core#26152)

`include/profile` and `lib/profile` have been added to compiler-rt since
emscripten-core#24160, but only `lib/profile` was added to `update_compiler_rt.py`,
which caused this directory to be missing from the subsequent LLVM
library updates.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants