merge issue on branch intrinsics-project

[Vicente Romero]

Hi,

I have pushed [1], which fixed the issue in the intrinsics repo,

Vicente

[1] http://hg.openjdk.java.net/amber/amber/rev/256be3515a30

On 4/22/19 1:24 PM, Vicente Romero wrote:
> Hi,
>
> There was an important changeset that was pushed last week, it was the 
> fix for [1], which changed the internal API used by javac to write to 
> the constant pool. This has provoked several merge issues in the amber 
> repos plus some build failures. We are working on fixing all the issues,
>
> Thanks,
> Vicente
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8222289
>
> On 4/20/19 6:29 AM, Remi Forax wrote:
>> Hi all,
>> the revision 18b81ba7105e of the branch intrinsics-project of 
>> amber/amber doesn't compile.
>>
>> /home/travis/build/forax/java-next/amber/src/jdk.compiler/share/classes/com/sun/tools/javac/comp/IntrinsicsVisitor.java:309: 
>> error: constructor DynamicMethodSymbol in class DynamicMethodSymbol 
>> cannot be applied to given types;
>>                      new DynamicMethodSymbol(methName,
>>                      ^
>>    required: Name,Symbol,MethodHandleSymbol,Type,LoadableConstant[]
>>    found: Name,TypeSymbol,int,MethodSymbol,MethodType,Object[]
>>    reason: actual and formal argument lists differ in length
>> 1 error
>> make[3]: *** 
>> [/home/travis/build/forax/java-next/amber/build/linux-x86_64-server-release/buildtools/interim_langtools_modules/jdk.compiler.interim/_the.BUILD_jdk.compiler.interim_batch] 
>> Error 1
>>
>> cheers,
>> Rémi
>