On Mon, Nov 26, 2018 at 11:36:43AM -0600, Kyrill Tkachov wrote: > Hi all, > > In the AAarch64 ISA the MUL and MNEG instructions are actually aliases of > MADD and MSUB. > Therefore they should have the type attribute mla, rather than mul, which > should only be used > for AArch32 32-bit multiplication instructions. > > This will ensure more consistent scheduling decisions. > > Bootstrapped and tested on aarch64-none-linux-gnu. > > Ok for trunk?
OK in principle. Did you audit the pipeline models to check this doesn't change scheduling class in an undesirable way for any of our supported targets? OK if so, if not can you run that audit and figure out the right thing to do to resolve it. Thanks, James