RFR: 8324679: Replace NULL with nullptr in HotSpot .ad files
Coleen Phillimore
coleenp at openjdk.org
Thu Feb 1 16:19:03 UTC 2024
On Thu, 1 Feb 2024 10:12:07 GMT, Andrew Haley <aph at openjdk.org> wrote:
>> Thanks for looking at this Vladimir. I made these changes. I don't know about this aarch64_ad.m4. It didn't have any NULL in it so not sure how it would be used to generate aarch64.ad. We can wait for Andrew to answer.
>>
>> For any other suggested changes, you can use a button like "suggest change" and I'll just click on it.
>
>> Thanks for looking at this Vladimir. I made these changes. I don't know about this aarch64_ad.m4. It didn't have any NULL in it so not sure how it would be used to generate aarch64.ad. We can wait for Andrew to answer.
>
> It's done manually. Just `m4 aarch64_ad.m4` then paste the result between
>
>
> // BEGIN This section of the file is automatically generated. Do not edit --------------
> // This section is generated from aarch64_ad.m4
>
>
> and
>
> `// END This section of the file is automatically generated. Do not edit --------------
> `
>
> Yea, that's a bit clunky, but it's a helluva lot more maintainable (and reliable!) than copy-and-pasting all of that boilerplate by hand.
Ok, I see in the .ad file. There are two sections contained within these BEGIN/END This section pairs. The sections didn't contain NULL since there wasn't any in the m4 file. So I didn't edit anything in these sections. Thanks @theRealAph
-------------
PR Comment: https://git.openjdk.org/jdk/pull/17658#issuecomment-1921693825
More information about the hotspot-compiler-dev
mailing list