On Mon, Mar 27, 2023 at 03:03:17PM +0800, Kewen.Lin wrote: > ... instead I suggested moving these three lines to below else arm for CCUNS, > since the arm for CC already has those variables redefined, so it's something > like:
I did those changes in the 3rd version of the patch. | Date: Mon, 27 Mar 2023 23:19:55 -0400 | From: Michael Meissner <meiss...@linux.ibm.com> | Subject: [PATCH, V3] PR target/105325, Make load/cmp fusion know about prefixed loads | Message-ID: <zcjc253l29auf...@toto.the-meissners.org> ... > In the previous review, I put a comment that "lp64 seems not necessary.". > Did you try to test without it? (if yes, any fallouts?) Yes, I tried it without the lp64, and I removed it from V3 of the patch. -- Michael Meissner, IBM PO Box 98, Ayer, Massachusetts, USA, 01432 email: meiss...@linux.ibm.com
