This is an archive of the discontinued LLVM Phabricator instance.

Differential D106101

[asan] Slightly modified the documentation.
ClosedPublic

Authored by kstoimenov on Jul 15 2021, 1:57 PM.

Details

Reviewers
vitalybuka
kcc
Commits
rG713949765614: [asan] Slightly modified the documentation.
Summary

The goal of this change is to test if I can commit changes.

Diff Detail

Unit TestsFailed

TimeTest
2,450 msx64 debian > libarcher.barrier::barrier.c
2,450 msx64 debian > libarcher.critical::critical.c
2,570 msx64 debian > libarcher.parallel::parallel-simple2.c
2,420 msx64 debian > libarcher.races::critical-unrelated.c
2,570 msx64 debian > libarcher.races::lock-nested-unrelated.c
View Full Test Results (18 Failed)

Event Timeline

kstoimenov requested review of this revision.Jul 15 2021, 1:57 PM
kstoimenov created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptJul 15 2021, 1:57 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
kstoimenov updated this revision to Diff 359129.Jul 15 2021, 2:00 PM
kstoimenov added a reviewer: kcc.

Add kcc as reviewer.

kstoimenov updated this revision to Diff 359130.Jul 15 2021, 2:01 PM
kstoimenov retitled this revision from [asan] Slightly modified the documentation to check commit permissions. to [asan] Slightly modified the documentation..
kstoimenov edited the summary of this revision. (Show Details)

Updated patch name.

Harbormaster completed remote builds in B114357: Diff 359130.Jul 15 2021, 4:14 PM
kstoimenov removed a reviewer: kcc.Jul 15 2021, 4:27 PM
kstoimenov added a reviewer: vitalybuka.
kcc accepted this revision.Jul 15 2021, 4:30 PM
kcc added a subscriber: kcc.

LGTM

This revision is now accepted and ready to land.Jul 15 2021, 4:30 PM
This revision was landed with ongoing or failed builds.Jul 15 2021, 4:36 PM
Closed by commit rG713949765614: [asan] Slightly modified the documentation. (authored by kstoimenov). · Explain Why
This revision was automatically updated to reflect the committed changes.
kstoimenov added a commit: rG713949765614: [asan] Slightly modified the documentation..

Revision Contents

PathSize
clang/
docs/
2 lines

Diff 359130

clang/docs/AddressSanitizer.rst

Show First 20 Lines • Show All 280 Lines • ▼ Show 20 Lines
Instrumentation code outlining Instrumentation code outlining
------------------------------ ------------------------------
By default AddressSanitizer inlines the instumentation code to improve the By default AddressSanitizer inlines the instumentation code to improve the
run-time performance, which leads to increased binary size. Using the run-time performance, which leads to increased binary size. Using the
(clang flag ``-fsanitize-address-outline-instrumentation` default: ``false``) (clang flag ``-fsanitize-address-outline-instrumentation` default: ``false``)
flag forces all code instumentation to be outlined, which reduces the size flag forces all code instumentation to be outlined, which reduces the size
of the binary, but also reduces the run-time performace. of the generated code, but also reduces the run-time performace.
Limitations Limitations
=========== ===========
* AddressSanitizer uses more real memory than a native run. Exact overhead * AddressSanitizer uses more real memory than a native run. Exact overhead
depends on the allocations sizes. The smaller the allocations you make the depends on the allocations sizes. The smaller the allocations you make the
bigger the overhead is. bigger the overhead is.
* AddressSanitizer uses more stack memory. We have seen up to 3x increase. * AddressSanitizer uses more stack memory. We have seen up to 3x increase.
Show All 34 Lines