Skip to content

Commit cbbdd62

Browse files
committed
address pr comments
1 parent 64937d2 commit cbbdd62

File tree

3 files changed

+3
-4
lines changed

3 files changed

+3
-4
lines changed

llvm/lib/Target/DirectX/DXILLegalizePass.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
//===- DXILLegalizePass.cpp - Legalizes llvm IR for DXIL-*- C++----------*-===//
1+
//===- DXILLegalizePass.cpp - Legalizes llvm IR for DXIL ------------------===//
22
//
33
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
44
// See https://llvm.org/LICENSE.txt for license information.
@@ -59,7 +59,7 @@ static bool fixI8TruncUseChain(Instruction &I,
5959
APInt Value = Imm->getValue();
6060
unsigned NewBitWidth = InstrType->getIntegerBitWidth();
6161
// Note: options here are sext or sextOrTrunc.
62-
// Since i8 isn't suppport we assume new values
62+
// Since i8 isn't supported, we assume new values
6363
// will always have a higher bitness.
6464
APInt NewValue = Value.sext(NewBitWidth);
6565
NewOperands.push_back(ConstantInt::get(InstrType, NewValue));

llvm/lib/Target/DirectX/DXILLegalizePass.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
//===- DXILLegalizePass.h - Legalizes llvm IR for DXIL-*- C++------------*-===//
1+
//===- DXILLegalizePass.h - Legalizes llvm IR for DXIL --------------------===//
22
//
33
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
44
// See https://llvm.org/LICENSE.txt for license information.

llvm/test/CodeGen/DirectX/legalize-i8.ll

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,6 @@ define i16 @i16_test(i16 %a) {
100100

101101
define i32 @all_imm() {
102102
; CHECK-LABEL: define i32 @all_imm(
103-
; CHECK-NOT: trunc
104103
; CHECK-NOT: sext i8
105104
; CHECK: ret i32 -1
106105
%1 = sub i8 0, 1

0 commit comments

Comments
 (0)