[llvm] c6f5f6a - [GlobalISel][X86] Move G_SEXT_INREG legalization handling to beside the regular integer extension legalizations

[Simon Pilgrim via llvm-commits]

Author: Simon Pilgrim
Date: 2023-06-06T14:39:10+01:00
New Revision: c6f5f6ab6197784613618acafdd26f121d2ca83d

URL: https://github.com/llvm/llvm-project/commit/c6f5f6ab6197784613618acafdd26f121d2ca83d
DIFF: https://github.com/llvm/llvm-project/commit/c6f5f6ab6197784613618acafdd26f121d2ca83d.diff

LOG: [GlobalISel][X86] Move G_SEXT_INREG legalization handling to beside the regular integer extension legalizations

Added: 
    

Modified: 
    llvm/lib/Target/X86/X86LegalizerInfo.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/X86/X86LegalizerInfo.cpp b/llvm/lib/Target/X86/X86LegalizerInfo.cpp
index d0611cd686fde..94250365ddbf6 100644
--- a/llvm/lib/Target/X86/X86LegalizerInfo.cpp
+++ b/llvm/lib/Target/X86/X86LegalizerInfo.cpp
@@ -290,6 +290,8 @@ X86LegalizerInfo::X86LegalizerInfo(const X86Subtarget &STI,
     .widenScalarToNextPow2(1, /*Min=*/8)
     .clampScalar(1, s8, sMaxScalar);
 
+  getActionDefinitionsBuilder(G_SEXT_INREG).lower();
+
   // fp constants
   getActionDefinitionsBuilder(G_FCONSTANT)
       .legalIf([=](const LegalityQuery &Query) -> bool {
@@ -376,8 +378,6 @@ void X86LegalizerInfo::setLegalizerInfo32bit() {
   // Control-flow
   LegacyInfo.setAction({G_BRCOND, s1}, LegacyLegalizeActions::Legal);
 
-  getActionDefinitionsBuilder(G_SEXT_INREG).lower();
-
   // Merge/Unmerge
   for (const auto &Ty : {s16, s32, s64}) {
     LegacyInfo.setAction({G_MERGE_VALUES, Ty}, LegacyLegalizeActions::Legal);