lenary wrote: Can we check what this callback is doing against the guidance here and the other targets: https://llvm.org/docs/InstCombineContributorGuide.html#canonicalization-and-target-independence
It's not clear to me how much we can take subtarget features into account when doing these combines. AArch64 seems to take into account the presence or absence of streaming mode in their checks (`instCombineSMECntsd`), so maybe this is equally ok for us to take into account the vlmin from target features. https://github.com/llvm/llvm-project/pull/218652 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
