This is an archive of the discontinued LLVM Phabricator instance.

tsan: de-hardcode MemCount const
ClosedPublic

Authored by dvyukov on Sep 27 2021, 4:31 AM.

Details

Summary

Use MemCount instead of hard-coded value 7.

Diff Detail

Event Timeline

dvyukov requested review of this revision.Sep 27 2021, 4:31 AM
dvyukov created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptSep 27 2021, 4:31 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
melver accepted this revision.Sep 27 2021, 4:57 AM
melver added inline comments.
compiler-rt/lib/tsan/rtl/tsan_platform_linux.cpp
125

This changes from 7 to 8 if I read the enum right.

I guess since the last argument of GetMemoryProfile is size, and mem was already MemCount, this is fine.

This revision is now accepted and ready to land.Sep 27 2021, 4:57 AM
This revision was landed with ongoing or failed builds.Sep 27 2021, 7:11 AM
This revision was automatically updated to reflect the committed changes.