RFR(S): 8238384: CTW: C2 compilation fails with "assert(store != load->find_exact_control(load->in(0))) failed: dependence cycle found"

[Tobias Hartmann]

Hi Roland,

On 21.02.20 10:30, Roland Westrelin wrote:
>> I agree with your analysis but I'm confused by your changes to Compile::can_alias(). You are
>> checking if the alias_idx belongs to a known instance address type but how can line 1909 then ever
>> return false? Doesn't alias_idx == AliasIdxBot imply known_instance == false?
> 
> Right. What about the following webrev?
> 
> http://cr.openjdk.java.net/~roland/8238384/webrev.01/

Looks good.

Why did webrev.00 work then?

I've noticed that you've removed the command lines from the test. No need anymore to run with
/othervm then. And you can remove the whitespace before the comment in compile.cpp:1907 (no new
webrev required) :)

Best regards,
Tobias