merge issue on branch intrinsics-project

[forax at univ-mlv.fr]

works for me,
thanks.

Rémi

----- Mail original -----
> De: "Vicente Romero" <vicente.romero at oracle.com>
> À: "Remi Forax" <forax at univ-mlv.fr>, "amber-dev" <amber-dev at openjdk.java.net>
> Envoyé: Lundi 22 Avril 2019 22:42:27
> Objet: Re: merge issue on branch intrinsics-project

> 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