Skip to content

Commit 660595f

Browse files
mmhammha
committed
Address more review comments
- Add header comments to CIROpInterfaces - Fix test clang-cir-ir-global
1 parent 817fc89 commit 660595f

File tree

4 files changed

+76
-64
lines changed

4 files changed

+76
-64
lines changed

‎clang/include/clang/CIR/Interfaces/CIROpInterfaces.h‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
66
//
77
//===----------------------------------------------------------------------===//
8+
//
9+
// Defines the interface to CIR operations.
10+
//
11+
//===----------------------------------------------------------------------===//
812

913
#ifndef CLANG_CIR_INTERFACES_CIR_OP_H
1014
#define CLANG_CIR_INTERFACES_CIR_OP_H

‎clang/include/clang/CIR/Interfaces/CIROpInterfaces.td‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
66
//
77
//===----------------------------------------------------------------------===//
8+
//
9+
// Defines the interface to CIR operations.
10+
//
11+
//===----------------------------------------------------------------------===//
812

913
#ifndef CLANG_CIR_INTERFACES_CIROPINTERFACES_TD
1014
#define CLANG_CIR_INTERFACES_CIROPINTERFACES_TD

‎clang/lib/CIR/Interfaces/CIROpInterfaces.cpp‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
66
//
77
//===----------------------------------------------------------------------===//
8+
//
9+
// Defines the interface to CIR operations.
10+
//
11+
//===----------------------------------------------------------------------===//
812
#include "clang/CIR/Interfaces/CIROpInterfaces.h"
913

1014
using namespace cir;

‎clang/test/CIR/IR/global.cir‎

Lines changed: 64 additions & 64 deletions
Original file line numberDiff line numberDiff line change
@@ -1,69 +1,69 @@
11
// RUN: cir-opt %s -o - | FileCheck %s
22

33
module attributes {cir.triple = "x86_64-unknown-linux-gnu"} {
4-
cir.global @c : !cir.int<s, 8>
5-
cir.global @sc : !cir.int<s, 8>
6-
cir.global @uc : !cir.int<u, 8>
7-
cir.global @ss : !cir.int<s, 16>
8-
cir.global @us = #cir.int<100> : !cir.int<u, 16>
9-
cir.global @si = #cir.int<42> : !cir.int<s, 32>
10-
cir.global @ui : !cir.int<u, 32>
11-
cir.global @sl : !cir.int<s, 64>
12-
cir.global @ul : !cir.int<u, 64>
13-
cir.global @sll : !cir.int<s, 64>
14-
cir.global @ull = #cir.int<123456> : !cir.int<u, 64>
15-
cir.global @s128 : !cir.int<s, 128>
16-
cir.global @u128 : !cir.int<u, 128>
17-
cir.global @wc : !cir.int<s, 32>
18-
cir.global @c8 : !cir.int<u, 8>
19-
cir.global @c16 : !cir.int<u, 16>
20-
cir.global @c32 : !cir.int<u, 32>
21-
cir.global @sb20 : !cir.int<s, 20>
22-
cir.global @ub48 : !cir.int<u, 48>
23-
cir.global @f16 : !cir.f16
24-
cir.global @bf16 : !cir.bf16
25-
cir.global @f : !cir.float
26-
cir.global @d = #cir.fp<1.250000e+00> : !cir.double
27-
cir.global @ld : !cir.long_double<!cir.f80>
28-
cir.global @f128 : !cir.f128
29-
cir.global @vp : !cir.ptr<!cir.void>
30-
cir.global @ip = #cir.ptr<null> : !cir.ptr<!cir.int<s, 32>>
31-
cir.global @dp : !cir.ptr<!cir.double>
32-
cir.global @cpp : !cir.ptr<!cir.ptr<!cir.int<s, 8>>>
33-
cir.global @fp : !cir.ptr<!cir.func<()>>
34-
cir.global @fpii = #cir.ptr<null> : !cir.ptr<!cir.func<(!cir.int<s, 32>) -> !cir.int<s, 32>>>
35-
cir.global @fpvar : !cir.ptr<!cir.func<(!cir.int<s, 32>, ...)>>
4+
cir.global external @c : !cir.int<s, 8>
5+
cir.global external @sc : !cir.int<s, 8>
6+
cir.global external @uc : !cir.int<u, 8>
7+
cir.global external @ss : !cir.int<s, 16>
8+
cir.global external @us = #cir.int<100> : !cir.int<u, 16>
9+
cir.global external @si = #cir.int<42> : !cir.int<s, 32>
10+
cir.global external @ui : !cir.int<u, 32>
11+
cir.global external @sl : !cir.int<s, 64>
12+
cir.global external @ul : !cir.int<u, 64>
13+
cir.global external @sll : !cir.int<s, 64>
14+
cir.global external @ull = #cir.int<123456> : !cir.int<u, 64>
15+
cir.global external @s128 : !cir.int<s, 128>
16+
cir.global external @u128 : !cir.int<u, 128>
17+
cir.global external @wc : !cir.int<s, 32>
18+
cir.global external @c8 : !cir.int<u, 8>
19+
cir.global external @c16 : !cir.int<u, 16>
20+
cir.global external @c32 : !cir.int<u, 32>
21+
cir.global external @sb20 : !cir.int<s, 20>
22+
cir.global external @ub48 : !cir.int<u, 48>
23+
cir.global external @f16 : !cir.f16
24+
cir.global external @bf16 : !cir.bf16
25+
cir.global external @f : !cir.float
26+
cir.global external @d = #cir.fp<1.250000e+00> : !cir.double
27+
cir.global external @ld : !cir.long_double<!cir.f80>
28+
cir.global external @f128 : !cir.f128
29+
cir.global external @vp : !cir.ptr<!cir.void>
30+
cir.global external @ip = #cir.ptr<null> : !cir.ptr<!cir.int<s, 32>>
31+
cir.global external @dp : !cir.ptr<!cir.double>
32+
cir.global external @cpp : !cir.ptr<!cir.ptr<!cir.int<s, 8>>>
33+
cir.global external @fp : !cir.ptr<!cir.func<()>>
34+
cir.global external @fpii = #cir.ptr<null> : !cir.ptr<!cir.func<(!cir.int<s, 32>) -> !cir.int<s, 32>>>
35+
cir.global external @fpvar : !cir.ptr<!cir.func<(!cir.int<s, 32>, ...)>>
3636
}
3737

38-
// CHECK: cir.global @c : !cir.int<s, 8>
39-
// CHECK: cir.global @sc : !cir.int<s, 8>
40-
// CHECK: cir.global @uc : !cir.int<u, 8>
41-
// CHECK: cir.global @ss : !cir.int<s, 16>
42-
// CHECK: cir.global @us = #cir.int<100>
43-
// CHECK: cir.global @si = #cir.int<42>
44-
// CHECK: cir.global @ui : !cir.int<u, 32>
45-
// CHECK: cir.global @sl : !cir.int<s, 64>
46-
// CHECK: cir.global @ul : !cir.int<u, 64>
47-
// CHECK: cir.global @sll : !cir.int<s, 64>
48-
// CHECK: cir.global @ull = #cir.int<123456> : !cir.int<u, 64>
49-
// CHECK: cir.global @s128 : !cir.int<s, 128>
50-
// CHECK: cir.global @u128 : !cir.int<u, 128>
51-
// CHECK: cir.global @wc : !cir.int<s, 32>
52-
// CHECK: cir.global @c8 : !cir.int<u, 8>
53-
// CHECK: cir.global @c16 : !cir.int<u, 16>
54-
// CHECK: cir.global @c32 : !cir.int<u, 32>
55-
// CHECK: cir.global @sb20 : !cir.int<s, 20>
56-
// CHECK: cir.global @ub48 : !cir.int<u, 48>
57-
// CHECK: cir.global @f16 : !cir.f16
58-
// CHECK: cir.global @bf16 : !cir.bf16
59-
// CHECK: cir.global @f : !cir.float
60-
// CHECK: cir.global @d = #cir.fp<1.250000e+00> : !cir.double
61-
// CHECK: cir.global @ld : !cir.long_double<!cir.f80>
62-
// CHECK: cir.global @f128 : !cir.f128
63-
// CHECK: cir.global @vp : !cir.ptr<!cir.void>
64-
// CHECK: cir.global @ip = #cir.ptr<null> : !cir.ptr<!cir.int<s, 32>>
65-
// CHECK: cir.global @dp : !cir.ptr<!cir.double>
66-
// CHECK: cir.global @cpp : !cir.ptr<!cir.ptr<!cir.int<s, 8>>>
67-
// CHECK: cir.global @fp : !cir.ptr<!cir.func<()>>
68-
// CHECK: cir.global @fpii = #cir.ptr<null> : !cir.ptr<!cir.func<(!cir.int<s, 32>) -> !cir.int<s, 32>>>
69-
// CHECK: cir.global @fpvar : !cir.ptr<!cir.func<(!cir.int<s, 32>, ...)>>
38+
// CHECK: cir.global external @c : !cir.int<s, 8>
39+
// CHECK: cir.global external @sc : !cir.int<s, 8>
40+
// CHECK: cir.global external @uc : !cir.int<u, 8>
41+
// CHECK: cir.global external @ss : !cir.int<s, 16>
42+
// CHECK: cir.global external @us = #cir.int<100>
43+
// CHECK: cir.global external @si = #cir.int<42>
44+
// CHECK: cir.global external @ui : !cir.int<u, 32>
45+
// CHECK: cir.global external @sl : !cir.int<s, 64>
46+
// CHECK: cir.global external @ul : !cir.int<u, 64>
47+
// CHECK: cir.global external @sll : !cir.int<s, 64>
48+
// CHECK: cir.global external @ull = #cir.int<123456> : !cir.int<u, 64>
49+
// CHECK: cir.global external @s128 : !cir.int<s, 128>
50+
// CHECK: cir.global external @u128 : !cir.int<u, 128>
51+
// CHECK: cir.global external @wc : !cir.int<s, 32>
52+
// CHECK: cir.global external @c8 : !cir.int<u, 8>
53+
// CHECK: cir.global external @c16 : !cir.int<u, 16>
54+
// CHECK: cir.global external @c32 : !cir.int<u, 32>
55+
// CHECK: cir.global external @sb20 : !cir.int<s, 20>
56+
// CHECK: cir.global external @ub48 : !cir.int<u, 48>
57+
// CHECK: cir.global external @f16 : !cir.f16
58+
// CHECK: cir.global external @bf16 : !cir.bf16
59+
// CHECK: cir.global external @f : !cir.float
60+
// CHECK: cir.global external @d = #cir.fp<1.250000e+00> : !cir.double
61+
// CHECK: cir.global external @ld : !cir.long_double<!cir.f80>
62+
// CHECK: cir.global external @f128 : !cir.f128
63+
// CHECK: cir.global external @vp : !cir.ptr<!cir.void>
64+
// CHECK: cir.global external @ip = #cir.ptr<null> : !cir.ptr<!cir.int<s, 32>>
65+
// CHECK: cir.global external @dp : !cir.ptr<!cir.double>
66+
// CHECK: cir.global external @cpp : !cir.ptr<!cir.ptr<!cir.int<s, 8>>>
67+
// CHECK: cir.global external @fp : !cir.ptr<!cir.func<()>>
68+
// CHECK: cir.global external @fpii = #cir.ptr<null> : !cir.ptr<!cir.func<(!cir.int<s, 32>) -> !cir.int<s, 32>>>
69+
// CHECK: cir.global external @fpvar : !cir.ptr<!cir.func<(!cir.int<s, 32>, ...)>>

0 commit comments

Comments
 (0)