[coreboot-gerrit] Change in coreboot[master]: lib: Add ASan support to ramstage on x86 arch

[Patrick Georgi (Code Review)]

Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42496 )

Change subject: lib: Add ASan support to ramstage on x86 arch
......................................................................

Patch Set 28:

(2 comments)

https://review.coreboot.org/c/coreboot/+/42496/10/src/lib/program.ld 
File src/lib/program.ld:

https://review.coreboot.org/c/coreboot/+/42496/10/src/lib/program.ld@131 
PS10, Line 131: /8
> I run into a linker error with this syntax (GCC 8.3.0, GNU ld (GNU Binutils) 2.30): […]

Some linker configurations take "/" as start of comment :-/

https://review.coreboot.org/c/coreboot/+/42496/19/src/lib/program.ld 
File src/lib/program.ld:

https://review.coreboot.org/c/coreboot/+/42496/19/src/lib/program.ld@130 
PS19, Line 130: .shadow . : {
              : 	_shadow_size = (_eheap - _data) >> 3;
              : 	. = ALIGN(ARCH_POINTER_ALIGN_SIZE);
              : 	_shadow = .;
              : 	. += _shadow_size;
              : 	. = ALIGN(ARCH_POINTER_ALIGN_SIZE);
              : 	_eshadow = .;
              : }
              : #endif
> How about: […]

Done