Put a pointer to linker script AST into the LinkingContext, so everyone can access linker script information.
This is the first step needed to implement semantic actions for Linker Script constructs.
Details
Details
- Reviewers
rafaelauler ruiu
Diff Detail
Diff Detail
Event Timeline
Comment Actions
I'd avoid carrying around raw linker script information as much as possible because it's too low level. An AST of a linker script is raw information about the linker script, and this patch will make it visible to virtually everyone (because the linking context object is shared by all components). It feels to me that this patch makes the information to be shared too broadly.
I made a change to interpret a linker script in the driver. I'd interpret linker scripts only in the driver and set cooked information to the linking context, instead of raw data. Is there any reason you can't do that way?