On 13/11/13 10:20, Joey Ye wrote: >>> + if (TARGET_VXWORKS_RTP) >>> > > + arm_pic_data_is_text_relative = 0; >> > >> > Why is this needed? Surely, even a VxWorks user should have the right to >> > force the compiler to behave differently. You've set things up through > the >> > default, now just accept what the user has asked for. > The reason is that TARGET_VXWORKS_RTP isn't a compile time value to initiate > arm.opt. Instead it is true only when -mrtp is specified in runtime. Also > enable text relative may result in runtime error on vxworks, it is better to > prevent it here.
I'd be happier if this was only done if the command-line option was not explicitly set on the command line. R.
