Dylan Tuttle

Results 18 comments of Dylan Tuttle

@keithc-ca @pshipton @zl-wang Now that the last warning has been disabled, could I get a review on this when one or more of you has a chance?

@keithc-ca How would I run a build of IBM Java 8 on AIX? That is the last build we have that still uses the xlC frontend and I would like...

Closed as completed in [omr #7461](https://github.com/eclipse/omr/pull/7461)

The ``` [ 18] O^O VALUE PROPAGATION: Removing rest of block after ArrayCopyBNDCHK [000000011F640230] ``` trace message comes from `OMR::ValuePropagation::mustTakeException()`. This function calls `OMR::ValuePropagation::removeRestOfBlock()` which actually performs the (seemingly) erroneous...

``` n1n BBStart (freq 10000) [ 0x11fb50190] bci=[-1,0,17] rc=0 vc=34 vn=- li=- udi=- nc=0 n4n istore [#352 Auto] [flags 0x3 0x0 ] [ 0x11fb50280] bci=[-1,2,17] rc=0 vc=34 vn=- li=- udi=-...

I believe I have found the source of the error! On line 1957 of [constrainAload()](https://github.com/eclipse/omr/blob/master/compiler/optimizer/VPHandlers.cpp#L1956-L1958): ```cpp if (elementSize != 0) { constraint = TR::VPClass::create(vp, (TR::VPClassType*)constraint, NULL, NULL, TR::VPArrayInfo::create(vp, 0, elementSize...

It looks like replacing the old method of calculating the length with ```cpp TR::Compiler->om.maxArraySizeInElements(elementSize, vp->comp()) ``` like in [`constrainArrayLength()`](https://github.com/eclipse/omr/blob/master/compiler/optimizer/VPHandlers.cpp#L4878) does fix the problem. Printing the values of `low` and `high`...