[llvm] [RISCV][GISel] Attempt to simplify how we handle type legality for F and D extensions. (PR #72174)
Michael Maitland via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 13 16:58:51 PST 2023
================
@@ -23,6 +23,15 @@
using namespace llvm;
using namespace LegalityPredicates;
+static LegalityPredicate typeIsLegalScalarFP(unsigned TypeIdx,
----------------
michaelmaitland wrote:
I prefer typeIsScalarFPArith to typeIsLegalScalarFP since it doesn't have the word `legal` in it. I think the word legal is unnecessary since it will go into a `legalIf` which makes it redundant. I like ScalarFPArith better than ScalarFP since it will suggest that not all ScalarFP instructions use it.
https://github.com/llvm/llvm-project/pull/72174
More information about the llvm-commits
mailing list