Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44235 )
Change subject: cpu/intel: Remove broken URLs ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44235/1/src/cpu/intel/model_67x/mod... File src/cpu/intel/model_67x/model_67x_init.c:
https://review.coreboot.org/c/coreboot/+/44235/1/src/cpu/intel/model_67x/mod... PS1, Line 39: * http://download.intel.com/design/intarch/specupdt/24445358.pdf
Please leave the document number in there.
I don't think it would be good for anything. Old document numbers aren't helpful when looking for the actual document, looking for the title name is probably easier. Moreover, newer CPUs don't have any comments at all. And as stated in the commit message, this is still available in git history.