RFR: 8171853: Remove Shark compiler

[Roman Kennke]

Hi David,

thanks!

I'm uploading a 2nd revision of the patch that excludes the 
generated-configure.sh part, and adds a smallish Zero-related fix.

http://cr.openjdk.java.net/~rkennke/8171853/webrev.01/ 
<http://cr.openjdk.java.net/%7Erkennke/8171853/webrev.01/>

Thanks, Roman


> Hi Roman,
>
> The build changes must be reviewed on build-dev - now cc'd.
>
> Thanks,
> David
>
> On 15/10/2017 8:41 AM, Roman Kennke wrote:
>> The JEP to remove the Shark compiler has received exclusively 
>> positive feedback (JDK-8189173) on zero-dev. So here comes the big 
>> patch to remove it.
>>
>> What I have done:
>>
>> grep -i -R shark src
>> grep -i -R shark make
>> grep -i -R shark doc
>> grep -i -R shark doc
>>
>> and purged any reference to shark. Almost everything was 
>> straightforward.
>>
>> The only things I wasn't really sure of:
>>
>> - in globals.hpp, I re-arranged the KIND_* bits to account for the 
>> gap that removing KIND_SHARK left. I hope that's good?
>> - in relocInfo_zero.hpp I put a ShouldNotCallThis() in 
>> pd_address_in_code(), I am not sure it is the right thing to do. If 
>> not, what *would* be the right thing?
>>
>> Then of course I did:
>>
>> rm -rf src/hotspot/share/shark
>>
>> I also went through the build machinery and removed stuff related to 
>> Shark and LLVM libs.
>>
>> Now the only references in the whole JDK tree to shark is a 'Shark 
>> Bay' in a timezone file, and 'Wireshark' in some tests ;-)
>>
>> I tested by building a regular x86 JVM and running JTREG tests. All 
>> looks fine.
>>
>> - I could not build zero because it seems broken because of the 
>> recent Atomic::* changes
>> - I could not test any of the other arches that seemed to reference 
>> Shark (arm and sparc)
>>
>> Here's the full webrev:
>>
>> http://cr.openjdk.java.net/~rkennke/8171853/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Erkennke/8171853/webrev.00/>
>>
>> Can I get a review on this?
>>
>> Thanks, Roman
>>