site/user/special/msan.md - Issue 1908493002: Add instructions for building with MSAN

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: site/user/special/msan.md

Issue 1908493002: Add instructions for building with MSAN (Closed) Base URL: https://skia.googlesource.com/skia.git@master

Patch Set: Created 4 years, 8 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

OLD	NEW
(Empty)
	1 Running with Memory Sanitizer
	scroggo 2016/04/20 14:39:10 I'm not very familiar with markdown, so the format I'm not very familiar with markdown, so the format is pretty basic. I'm open to suggestions. The instructions are also pretty basic. They are borrowed from crrev.com/1665823002. hal.canary 2016/04/20 14:45:43 The msan build builds Clang from scratch, so you n Show quoted text On 2016/04/20 14:39:10, scroggo wrote: > I'm not very familiar with markdown, so the format is pretty basic. I'm open to > suggestions. > > The instructions are also pretty basic. They are borrowed from > crrev.com/1665823002. The msan build builds Clang from scratch, so you need to download Clang first: Add the following to your gclient file: target_os = ['llvm'] Then run: gclient sync Build ----- out/Debug/dm -v --match ~Codec ~BlurLargeImage ~FontMgrAndroidParser Run --- out/Debug/dm -v --match ~Codec ~BlurLargeImage ~FontMgrAndroidParser scroggo 2016/04/20 15:02:19 It looks like you're recommending - four spaces be Show quoted text On 2016/04/20 14:45:43, Hal Canary wrote: > On 2016/04/20 14:39:10, scroggo wrote: > > I'm not very familiar with markdown, so the format is pretty basic. I'm open > to > > suggestions. > > > > The instructions are also pretty basic. They are borrowed from > > crrev.com/1665823002. > > The msan build builds Clang from scratch, so you need to download Clang first: > > Add the following to your gclient file: > > target_os = ['llvm'] > > Then run: > > gclient sync > > Build > ----- > > out/Debug/dm -v --match ~Codec ~BlurLargeImage ~FontMgrAndroidParser > Run > --- > > out/Debug/dm -v --match ~Codec ~BlurLargeImage ~FontMgrAndroidParser It looks like you're recommending - four spaces before commands - no "$" - no "./" ? Done. FWIW, our other docs don't seem to be consistent about using a "$".
	2 =============================

	3

	4 Prerequisites

	5 -------------

	6

	7 The msan build builds Clang from scratch, so you need to download Clang first:

	8

	9 Add the following to your gclient file:

	10 target_os = ['llvm']

	11

	12 Then run:

	13 $ gclient sync
	hal.canary 2016/04/20 14:48:11 Run it this way instead: bin/sync-and-gyp --d Run it this way instead: bin/sync-and-gyp --deps=llvm scroggo 2016/04/20 15:02:19 Done. Show quoted text On 2016/04/20 14:48:11, Hal Canary wrote: > Run it this way instead: > > bin/sync-and-gyp --deps=llvm Done.
	14

	15 Build

	16 -----

	17

	18 $ ./tools/xsan_build memory dm

	19

	20 Run

	21 ---

	22

	23 $ out/Debug/dm -v --match ~Codec ~BlurLargeImage ~FontMgrAndroidParser

OLD	NEW

« no previous file with comments | « no previous file | tests/ImageTest.cpp » ('j') | no next file with comments »