From HORIE at jp.ibm.com Tue Aug 18 08:38:42 2020 From: HORIE at jp.ibm.com (Michihiro Horie) Date: Tue, 18 Aug 2020 17:38:42 +0900 Subject: RFR: 8251926: PPC: Remove an unused variable in assembler_ppc.cpp Message-ID: Dear all, Would you please review a small change? Bug: https://bugs.openjdk.java.net/browse/JDK-8251926 Webrev: http://cr.openjdk.java.net/~mhorie/8251926/webrev.00/ The load_const_optimized function in assembler_ppc.cpp has an unused variable named return_xd. It looks unnecessary in the current code. Best regards, Michihiro -------------- next part -------------- An HTML attachment was scrubbed... URL: From thomas.stuefe at gmail.com Tue Aug 18 09:23:04 2020 From: thomas.stuefe at gmail.com (=?UTF-8?Q?Thomas_St=C3=BCfe?=) Date: Tue, 18 Aug 2020 11:23:04 +0200 Subject: RFR: 8251926: PPC: Remove an unused variable in assembler_ppc.cpp In-Reply-To: References: Message-ID: Hi Michihiro, seems fine and trivial. Thanks, Thomas On Tue, Aug 18, 2020 at 10:40 AM Michihiro Horie wrote: > Dear all, > > Would you please review a small change? > > Bug: https://bugs.openjdk.java.net/browse/JDK-8251926 > Webrev: http://cr.openjdk.java.net/~mhorie/8251926/webrev.00/ > > The load_const_optimized function in assembler_ppc.cpp has an unused > variable named return_xd. It looks unnecessary in the current code. > > Best regards, > Michihiro > -------------- next part -------------- An HTML attachment was scrubbed... URL: From HORIE at jp.ibm.com Tue Aug 18 09:49:19 2020 From: HORIE at jp.ibm.com (Michihiro Horie) Date: Tue, 18 Aug 2020 18:49:19 +0900 Subject: RFR: 8251926: PPC: Remove an unused variable in assembler_ppc.cpp In-Reply-To: References: , Message-ID: Hi Thomas, Thanks a lot! Best regards, Michihiro ----- Original message ----- From: "Thomas St?fe" To: Michihiro Horie Cc: ppc-aix-port-dev , hotspot compiler Subject: [EXTERNAL] Re: RFR: 8251926: PPC: Remove an unused variable in assembler_ppc.cpp Date: Tue, Aug 18, 2020 6:23 PM Hi Michihiro, seems fine and trivial. Thanks, Thomas On Tue, Aug 18, 2020 at 10:40 AM Michihiro Horie wrote: Dear all, Would you please review a small change? Bug: https://bugs.openjdk.java.net/browse/JDK-8251926 Webrev: http://cr.openjdk.java.net/~mhorie/8251926/webrev.00/ The load_const_optimized function in assembler_ppc.cpp has an unused variable named return_xd. It looks unnecessary in the current code. Best regards, Michihiro -------------- next part -------------- An HTML attachment was scrubbed... URL: