diff options
| author | David Green <david.green@arm.com> | 2018-02-01 13:05:25 +0000 |
|---|---|---|
| committer | David Green <david.green@arm.com> | 2018-02-01 13:05:25 +0000 |
| commit | 184df0c35d9a050590aacc6e4862ef1aec125a34 (patch) | |
| tree | 12f7d635e5db3fecc0d5e60dce4238bbe9342959 /llvm/lib/Transforms | |
| parent | a330c208f28d6522fff18b4ac2ca6db5b93e83f2 (diff) | |
| download | bcm5719-llvm-184df0c35d9a050590aacc6e4862ef1aec125a34.tar.gz bcm5719-llvm-184df0c35d9a050590aacc6e4862ef1aec125a34.zip | |
Revert commit rL323951
Looks like it's causing timeouts out on at least ppc64le
buildbots.
llvm-svn: 323959
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/InstCombine/InstructionCombining.cpp | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp b/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp index 47e87a20c92..b332e75c7fe 100644 --- a/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp +++ b/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp @@ -144,15 +144,11 @@ Value *InstCombiner::EmitGEPOffset(User *GEP) { /// We don't want to convert from a legal to an illegal type or from a smaller /// to a larger illegal type. A width of '1' is always treated as a legal type /// because i1 is a fundamental type in IR, and there are many specialized -/// optimizations for i1 types. Widths of 8, 16 or 32 are equally treated as -/// legal to convert to, in order to open up more combining opportunities. -/// NOTE: this treats i8, i16 and i32 specially, due to them being so common -/// from frontend languages. +/// optimizations for i1 types. bool InstCombiner::shouldChangeType(unsigned FromWidth, unsigned ToWidth) const { bool FromLegal = FromWidth == 1 || DL.isLegalInteger(FromWidth); - bool ToLegal = ToWidth == 1 || ToWidth == 8 || ToWidth == 16 || - ToWidth == 32 || DL.isLegalInteger(ToWidth); + bool ToLegal = ToWidth == 1 || DL.isLegalInteger(ToWidth); // If this is a legal integer from type, and the result would be an illegal // type, don't do the transformation. |
