[llvm] r309090 - [ARM] GlobalISel: Mark G_GLOBAL_VALUE as legal

[Diana Picus via llvm-commits]

Author: rovka
Date: Wed Jul 26 02:25:15 2017
New Revision: 309090

URL: http://llvm.org/viewvc/llvm-project?rev=309090&view=rev
Log:
[ARM] GlobalISel: Mark G_GLOBAL_VALUE as legal

Modified:
    llvm/trunk/lib/Target/ARM/ARMLegalizerInfo.cpp
    llvm/trunk/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir

Modified: llvm/trunk/lib/Target/ARM/ARMLegalizerInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/ARMLegalizerInfo.cpp?rev=309090&r1=309089&r2=309090&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/ARMLegalizerInfo.cpp (original)
+++ llvm/trunk/lib/Target/ARM/ARMLegalizerInfo.cpp Wed Jul 26 02:25:15 2017
@@ -43,6 +43,7 @@ ARMLegalizerInfo::ARMLegalizerInfo(const
   const LLT s32 = LLT::scalar(32);
   const LLT s64 = LLT::scalar(64);
 
+  setAction({G_GLOBAL_VALUE, p0}, Legal);
   setAction({G_FRAME_INDEX, p0}, Legal);
 
   for (unsigned Op : {G_LOAD, G_STORE}) {

Modified: llvm/trunk/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir?rev=309090&r1=309089&r2=309090&view=diff
==============================================================================
--- llvm/trunk/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir (original)
+++ llvm/trunk/test/CodeGen/ARM/GlobalISel/arm-legalizer.mir Wed Jul 26 02:25:15 2017
@@ -47,6 +47,9 @@
   define void @test_fadd_s32() #0 { ret void }
   define void @test_fadd_s64() #0 { ret void }
 
+  @a_global = global i32 42
+  define void @test_global_variable() { ret void }
+
   attributes #0 = { "target-features"="+vfp2" }
 ...
 ---
@@ -948,3 +951,26 @@ body:             |
     BX_RET 14, _, implicit %d0
 
 ...
+---
+name:            test_global_variable
+# CHECK-LABEL: name: test_global_variable
+legalized:       false
+# CHECK: legalized: true
+regBankSelected: false
+selected:        false
+tracksRegLiveness: true
+registers:
+  - { id: 0, class: _ }
+  - { id: 1, class: _ }
+body:             |
+  bb.0:
+    liveins: %r0
+
+    %0(s32) = COPY %r0
+    %1(p0) = G_GLOBAL_VALUE @a_global
+    ; G_GLOBAL_VALUE is legal, so we should find it unchanged in the output
+    ; CHECK: {{%[0-9]+}}(p0) = G_GLOBAL_VALUE @a_global
+    %r0 = COPY %1(p0)
+    BX_RET 14, _, implicit %r0
+
+...