This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer] Add symbolizer support for loongarch64
ClosedPublic

Authored by tangyouling on Nov 3 2022, 7:46 PM.

Diff Detail

Event Timeline

tangyouling created this revision.Nov 3 2022, 7:46 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 3 2022, 7:46 PM
tangyouling requested review of this revision.Nov 3 2022, 7:46 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 3 2022, 7:46 PM
Herald added subscribers: Restricted Project, pcwang-thead. · View Herald Transcript
xry111 accepted this revision.Nov 3 2022, 7:49 PM

Straightforward enough to me.

This revision is now accepted and ready to land.Nov 3 2022, 7:49 PM
xen0n added inline comments.Nov 3 2022, 11:34 PM
compiler-rt/lib/sanitizer_common/sanitizer_common.h
712

Should use the proper casing LoongArch. See Hexagon below; our name is not an acronym so don't make it all-caps.

"LOONGARCH64" -> "LoongArch64"

This revision was landed with ongoing or failed builds.Nov 9 2022, 9:38 PM
This revision was automatically updated to reflect the committed changes.