mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-23 19:23:23 +01:00
c5d6ea096a
Summary: When comparing basic blocks, there is an additional check that two Value*'s should have the same ID, which interferes with merging equivalent constants of different kinds (such as a ConstantInt and a ConstantPointerNull in the included testcase). The cmpValues function already ensures that the two values in each function are the same, so removing this check should not cause incorrect merging. Also, the type comparison is redundant, based on reviewing the code and testing on the test suite and several large LTO bitcodes. Author: jrkoenig Reviewers: nlewycky, jfb, dschuff Subscribers: llvm-commits Differential revision: http://reviews.llvm.org/D12302 llvm-svn: 246001
24 lines
680 B
LLVM
24 lines
680 B
LLVM
; RUN: opt -mergefunc -S < %s | FileCheck %s
|
|
target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128-n8:16:32:64-S128"
|
|
|
|
; These should not be merged, as 1 != 0.
|
|
define internal i64 @Ifunc(i32* %P, i32* %Q) {
|
|
; CHECK-LABEL: define internal i64 @Ifunc
|
|
; CHECK-NEXT: store
|
|
; CHECK-NEXT: store
|
|
; CHECK-NEXT: ret
|
|
store i32 10, i32* %P
|
|
store i32 10, i32* %Q
|
|
ret i64 1
|
|
}
|
|
|
|
define internal i64* @Jfunc(i32* %P, i32* %Q) {
|
|
; CHECK-LABEL: define internal i64* @Jfunc
|
|
; CHECK-NEXT: store
|
|
; CHECK-NEXT: store
|
|
; CHECK-NEXT: ret
|
|
store i32 10, i32* %P
|
|
store i32 10, i32* %Q
|
|
ret i64* null
|
|
}
|