This is an archive of the discontinued LLVM Phabricator instance.

[profile] Add underscore to /alternatename for Win/x86
ClosedPublic

Authored by aeubanks on Jul 28 2021, 1:37 PM.

Details

Summary

/alternatename should use the mangled name. On x86 we need an extra
underscore.

Copied from sanitizer_win_defs.h

Fixes https://crbug.com/1233589.

Diff Detail

Event Timeline

aeubanks requested review of this revision.Jul 28 2021, 1:37 PM
aeubanks created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJul 28 2021, 1:37 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
aeubanks edited the summary of this revision. (Show Details)Jul 28 2021, 1:39 PM

not really sure how to add a test for this

phosek accepted this revision.Jul 28 2021, 2:21 PM

LGTM thanks for catching this! This is another reason to do https://lists.llvm.org/pipermail/llvm-dev/2021-July/151789.html since if we just used WIN_WEAK_ALIAS we could have avoid this issue.

compiler-rt/lib/profile/InstrProfilingFile.c
597–606

Could we reverse the condition just like what sanitizer_win_defs.h does.

This revision is now accepted and ready to land.Jul 28 2021, 2:21 PM
aeubanks added inline comments.Jul 28 2021, 2:43 PM
compiler-rt/lib/profile/InstrProfilingFile.c
597–606

I can do that, but this was copied directly from sanitizer_win_defs.h, sanitizer_win_defs.h doesn't use _WIN64

phosek added inline comments.Jul 28 2021, 2:54 PM
compiler-rt/lib/profile/InstrProfilingFile.c
597–606

Never mind, I was looking at older version but it looks like sanitizer_win_defs.h switched to this version to support MinGW.

This revision was landed with ongoing or failed builds.Jul 28 2021, 2:59 PM
This revision was automatically updated to reflect the committed changes.