This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] fix real strlens that were missed
ClosedPublic

Authored by michaelrj on Aug 27 2021, 1:57 PM.

Details

Summary

In reviews.llvm.org/D108316 I missed a few instances of REAL(strlen) and
this change fixes that, as well as restoring one that seems like it
shouldn't have been changed.

Diff Detail

Event Timeline

michaelrj created this revision.Aug 27 2021, 1:57 PM
michaelrj requested review of this revision.Aug 27 2021, 1:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 27 2021, 1:57 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
hctim added inline comments.Aug 27 2021, 1:58 PM
compiler-rt/lib/sanitizer_common/sanitizer_common_interceptors.inc
7061

i think this should be the other way around, REAL(wcslen) should become internal_wcslen here.

michaelrj updated this revision to Diff 369194.Aug 27 2021, 2:43 PM
michaelrj marked an inline comment as done.

switch RealStrLen to use internal_strlen and internal_wcslen

vitalybuka added inline comments.
compiler-rt/lib/sanitizer_common/sanitizer_common_interceptors.inc
7061

internal_wcslen

vitalybuka accepted this revision.Aug 27 2021, 2:47 PM

LGTM

compiler-rt/lib/sanitizer_common/sanitizer_common_interceptors.inc
7061

it's done already.

This revision is now accepted and ready to land.Aug 27 2021, 2:47 PM
hctim accepted this revision.Aug 27 2021, 2:47 PM

LGTM

This revision was landed with ongoing or failed builds.Aug 27 2021, 3:33 PM
This revision was automatically updated to reflect the committed changes.