[PATCH 2/2] rust: update usage of TARGET_AIX to TARGET_AIX_OS
Paul E Murphy
murphyp@linux.ibm.com
Wed Jun 21 16:20:46 GMT 2023
On 6/19/23 3:39 AM, Thomas Schwinge wrote:
> Hi Paul!
>
> On 2023-06-16T11:00:02-0500, "Paul E. Murphy via Gcc-patches" <gcc-patches@gcc.gnu.org> wrote:
>> This was noticed when fixing the gccgo usage of the macro, the
>> rust usage is very similar.
>>
>> TARGET_AIX is defined as a non-zero value on linux/powerpc64le
>> which may cause unexpected behavior. TARGET_AIX_OS should be
>> used to toggle AIX specific behavior.
>>
>> gcc/rust/ChangeLog:
>>
>> * rust-object-export.cc [TARGET_AIX]: Rename and update
>> usage to TARGET_AIX_OS.
>
> I don't have rights to formally approve this GCC/Rust change, but I'll
> note that it follows "as obvious" (see
> <https://gcc.gnu.org/gitwrite.html#policies>, "Obvious fixes") to the
> corresponding GCC/Go change, which has been approved:
> <https://inbox.sourceware.org/CAKOQZ8wDWc7G5_jBNK1jvgChhiURceeAmZb5Bqrx_VZjeJP2Vw@mail.gmail.com>,
> and which is where this GCC/Rust code has been copied from, so I suggest
> you push both patches at once.
>
>
> Grüße
> Thomas
Hi Thomas,
Thank you for reviewing. I do not have commit access, so I cannot push
this myself. If this is OK, could one of the rust maintainers push this
patch?
Thanks,
Paul
More information about the Gcc-rust
mailing list