RFR: Refactor asm acmp

Andrew Haley aph at redhat.com
Tue Jul 18 16:40:24 UTC 2017


On 18/07/17 17:17, Roman Kennke wrote:
> Am 18.07.2017 um 18:14 schrieb Andrew Haley:
>> On 18/07/17 17:00, Aleksey Shipilev wrote:
>>> On 07/18/2017 05:57 PM, Roman Kennke wrote:
>>>> Oops, some dead stuff sneaked into shenandoahBarrierSet.hpp. Removed:
>>>>
>>>> http://cr.openjdk.java.net/~rkennke/refactor-acmp/webrev.02/
>>> Looks much nicer, go. Only x86, not AArch64?
>> cmpoopptr is a strange name: I'd read it as comparing pointers to oops,
>> not comparing oops.  Which does it actually do?
>>
> it compares oops with oops. cmpoop() is already taken (although we'll
> probably do some refactoring and reuse it later, but it requires some
> upstream work).

cmp_oop_oop ?  Makes much more sense to this reader.


-- 
Andrew Haley
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671


More information about the shenandoah-dev mailing list