[PATCH] D47424: [MIRParser] Add parser support for 'true' and 'false' i1s.

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 2 04:13:42 PDT 2018


aemerson updated this revision to Diff 149607.

Repository:
  rL LLVM

https://reviews.llvm.org/D47424

Files:
  lib/CodeGen/MIRParser/MIParser.cpp
  test/CodeGen/MIR/Generic/parse-integer-true-false.mir


Index: test/CodeGen/MIR/Generic/parse-integer-true-false.mir
===================================================================
--- /dev/null
+++ test/CodeGen/MIR/Generic/parse-integer-true-false.mir
@@ -0,0 +1,11 @@
+# RUN: llc -run-pass none -o - %s | FileCheck %s
+# Parse an i1 being a 'true' or 'false'
+---
+name: i1_true_false
+body: |
+  bb.0:
+    ; CHECK: %0:_(s1) = G_CONSTANT i1 true
+    ; CHECK: %1:_(s1) = G_CONSTANT i1 false
+    %0:_(s1) = G_CONSTANT i1 true
+    %1:_(s1) = G_CONSTANT i1 false
+...
Index: lib/CodeGen/MIRParser/MIParser.cpp
===================================================================
--- lib/CodeGen/MIRParser/MIParser.cpp
+++ lib/CodeGen/MIRParser/MIParser.cpp
@@ -1373,8 +1373,11 @@
 
   auto Loc = Token.location();
   lex();
-  if (Token.isNot(MIToken::IntegerLiteral))
-    return error("expected an integer literal");
+  if (Token.isNot(MIToken::IntegerLiteral)) {
+    if (Token.isNot(MIToken::Identifier) ||
+        !(Token.range() == "true" || Token.range() == "false"))
+      return error("expected an integer literal");
+  }
   const Constant *C = nullptr;
   if (parseIRConstant(Loc, C))
     return true;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47424.149607.patch
Type: text/x-patch
Size: 1164 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180602/85562d11/attachment.bin>


More information about the llvm-commits mailing list