diff options
| author | Michael Zolotukhin <mzolotukhin@apple.com> | 2014-05-05 14:46:53 +0000 |
|---|---|---|
| committer | Michael Zolotukhin <mzolotukhin@apple.com> | 2014-05-05 14:46:53 +0000 |
| commit | 4e030e8fb42f4749fd759daa42ecf43712e4f25b (patch) | |
| tree | b8f9c76be177acef964a1a56a75da534334ed587 | |
| parent | 9dbc20630358c71402527618e6e381efe3906599 (diff) | |
| download | bcm5719-llvm-4e030e8fb42f4749fd759daa42ecf43712e4f25b.tar.gz bcm5719-llvm-4e030e8fb42f4749fd759daa42ecf43712e4f25b.zip | |
Fix test from r207966 and add a comment there.
llvm-svn: 207969
| -rw-r--r-- | llvm/test/Transforms/ConstantHoisting/X86/large-immediate-2.ll | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/llvm/test/Transforms/ConstantHoisting/X86/large-immediate-2.ll b/llvm/test/Transforms/ConstantHoisting/X86/large-immediate-2.ll index 5f23e0a3299..883be355bd3 100644 --- a/llvm/test/Transforms/ConstantHoisting/X86/large-immediate-2.ll +++ b/llvm/test/Transforms/ConstantHoisting/X86/large-immediate-2.ll @@ -1,8 +1,10 @@ ; RUN: llc < %s -O3 -march=x86-64 |FileCheck %s define i64 @foo(i1 %z, i192* %p, i192* %q) { -; CHECK: movq 16(%rsi), %rax -; CHECK-NEXT: retq +; If const 128 is hoisted to a variable, then in basic block L_val2 we would +; have %lshr2 = lshr i192 %data2, %const, and the definition of %const would +; be in another basic block. As a result, a very inefficient code might be +; produced. Here we check that this doesn't occur. entry: %data1 = load i192* %p, align 8 %lshr1 = lshr i192 %data1, 128 |
