[PATCH] D14803: [AArch64] Fix a corner case in BitFeild select
Mandeep Singh Grang via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 18 22:30:22 PST 2015
mgrang added a comment.
Please fix the typo in the commit title:
BitFeild ==> BitField
================
Comment at: lib/Target/AArch64/AArch64ISelDAGToDAG.cpp:1699
@@ -1698,1 +1698,3 @@
+ // trivially true.
+ if (BitWidth == 0)
----------------
Comments should begin with an uppercase letter.
================
Comment at: lib/Target/AArch64/AArch64ISelDAGToDAG.cpp:1700
@@ +1699,3 @@
+ // trivially true.
+ if (BitWidth == 0)
+ return true;
----------------
Can we change this to if (!BitWidth) ?
================
Comment at: test/CodeGen/AArch64/bitfield-pr25571.ll:2
@@ +1,3 @@
+; ModuleID = 'bugpoint-reduced-simplified.bc'
+;RUN: llc -verify-machineinstrs < %s | FileCheck %s
+target datalayout = "e-m:e-i64:64-i128:128-n8:16:32:64-S128"
----------------
Space between ;RUN
http://reviews.llvm.org/D14803
More information about the llvm-commits
mailing list