RFR: 8325821: [REDO] use "dmb.ishst+dmb.ishld" for release barrier [v3]
kuaiwei
duke at openjdk.org
Tue Apr 9 07:15:09 UTC 2024
On Mon, 8 Apr 2024 17:36:42 GMT, Andrew Haley <aph at openjdk.org> wrote:
> That doesn't look bad. I've made a bunch of simplifications for you to see: have a look. In general, pointer chasing is bad, so it's worth getting rid of indirections that don't help. [foo.zip](https://github.com/openjdk/jdk/files/14908996/foo.zip)
Thanks for your enhancement. I found you removed MergeableInst. My idea is the finite state machine could support both dmb and ld/st instructions. I created another task to merge more instruction like 'ldrs/ldrd/strs/strd ...' . https://bugs.openjdk.org/browse/JDK-8329901 . The old implementation is not suitable to support new instruction. I need duplicate to support other register type like float register. So I want to use MergeableInst to extract instruction information. Such as target register, base register, offseet ... . It's my draft plan, how do you think about it?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/18467#issuecomment-2044294247
More information about the hotspot-dev
mailing list