[llvm] r323377 - [InstCombine] fix datalayout in test file

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 24 13:36:45 PST 2018


Author: spatel
Date: Wed Jan 24 13:36:45 2018
New Revision: 323377

URL: http://llvm.org/viewvc/llvm-project?rev=323377&view=rev
Log:
[InstCombine] fix datalayout in test file

The only part of the datalayout that should matter for these tests
is the part that specifies the legal int widths ('n*'). But there
was a bug - that part of the string was not correctly separated with
the expected '-' character, so we were testing as if there were no
legal int widths at all. Removed the leading cruft so we have some 
legal ints to test with.

I noticed this while testing a potential change to the way we 
transform shifts and sexts in D42424.

Modified:
    llvm/trunk/test/Transforms/InstCombine/signext.ll

Modified: llvm/trunk/test/Transforms/InstCombine/signext.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/InstCombine/signext.ll?rev=323377&r1=323376&r2=323377&view=diff
==============================================================================
--- llvm/trunk/test/Transforms/InstCombine/signext.ll (original)
+++ llvm/trunk/test/Transforms/InstCombine/signext.ll Wed Jan 24 13:36:45 2018
@@ -1,7 +1,7 @@
 ; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
 ; RUN: opt < %s -instcombine -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"
+target datalayout = "n8:16:32:64"
 
 define i32 @test1(i32 %x) {
 ; CHECK-LABEL: @test1(




More information about the llvm-commits mailing list