[GOLD][PATCH] Remove Output_section::Simple_input_section class.
Ian Lance Taylor
iant@google.com
Wed May 19 06:13:00 GMT 2010
"Doug Kwan (關振德)" <dougkwan@google.com> writes:
> 在 2010年5月19日上午11:49,Ian Lance Taylor <iant@google.com> 寫道:
>
>>> @@ -1215,7 +1219,7 @@ class Arm_input_section : public Output_
>>> // Address alignment of the original input section.
>>> uint64_t original_addralign_;
>>> // Section size of the original input section.
>>> - uint64_t original_size_;
>>> + off_t original_size_;
>>
>> This change doesn't seem quite right. Why should the size of a
>> section be represented by an off_t value? off_t is for file offsets,
>> not sizes. On the other hand, since this is ARM which is known to be
>> a 32-bit target, it would be fine to use uint32_t here.
>
> We use off_t for data_size() in Output_data and its sub-classes. I
> changed this to make it consistent. I agree that uint32_t is
> sufficient for ARM and will change this.
Oh yeah. That dates back to when I created output.h. It should
probably change at some point.
Ian
More information about the Binutils
mailing list