[PATCH] [lld] Make ELFLinkingContext own LinkerScript buffers

Rui Ueyama ruiu at google.com
Sun Feb 1 15:01:36 PST 2015


I didn't think hard enough about that, but it might be good to have a class to cook linker scripts. But if you have such class, you can instantiate that class in the driver and only set that class instance to the linking context, no? You wouldn't have to add a pointer to linker script ASTs to the linking context, I guess.

I prefer a patch that does one thing; in this case, we should try to fix the unsafe memory access first, so I'd fix that without adding any new feature.


http://reviews.llvm.org/D7323

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list