This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer] Replace a local array with InternalScopedString in MaybeReexec()
ClosedPublic

Authored by kubamracek on Dec 3 2015, 3:02 AM.

Details

Summary

MaybeReexec contains a 1024-byte long local array, which produces a warning about frame size:

.../lib/sanitizer_common/sanitizer_mac.cc:548:6: warning: stack frame size of 1132 bytes in function '__sanitizer::MaybeReexec' [-Wframe-larger-than=]

Let's replace it with InternalScopedString.

Diff Detail

Event Timeline

kubamracek updated this revision to Diff 41732.Dec 3 2015, 3:02 AM
kubamracek retitled this revision from to [sanitizer] Replace a local array with InternalScopedString in MaybeReexec().
kubamracek updated this object.
kubamracek added reviewers: dvyukov, samsonov, glider, kcc.
kubamracek added subscribers: llvm-commits, zaks.anna.
dvyukov accepted this revision.Dec 3 2015, 4:48 AM
dvyukov edited edge metadata.
This revision is now accepted and ready to land.Dec 3 2015, 4:48 AM
This revision was automatically updated to reflect the committed changes.