Harshit Sharma has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44814 )
Change subject: Documentation: Add ASan documentation ......................................................................
Patch Set 7:
(3 comments)
Thanks for proofreading. Let me know if any further corrections or additions are required.
https://review.coreboot.org/c/coreboot/+/44814/6/Documentation/technotes/asa... File Documentation/technotes/asan.md:
https://review.coreboot.org/c/coreboot/+/44814/6/Documentation/technotes/asa... PS6, Line 156: Let's subtract the : start address of the stage from `ip`
For this, the start address of the stage is missing in your text, or? This address depends on where […]
I've now mentioned the start address at line 137.
https://review.coreboot.org/c/coreboot/+/44814/6/Documentation/technotes/asa... PS6, Line 205: we
We (Start uppercase after the "Voilà!"?)
Done
https://review.coreboot.org/c/coreboot/+/44814/6/Documentation/technotes/asa... PS6, Line 298:
Extra white space.
Removed.