Page MenuHomePhabricator

Added checks for BinaryOperation, InsertElement, ExtractElement and changed return value in FunctionComparator
Needs ReviewPublic

Authored by vish99 on Jun 7 2020, 12:22 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Added special checks for BinaryOperator, InsertElement, ExtractElement since the return value is changed to return the left operation is greater if the operation opcode is not seen so far. This will not return unknown operations as equal.

Diff Detail

Unit TestsFailed

TimeTest
3,390 msLLVM.Transforms/MergeFunc::Unknown Unit Message ("")
Script: -- : 'RUN: at line 1'; /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/build/bin/opt -mergefunc -disable-output < /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/llvm/test/Transforms/MergeFunc/2011-02-08-RemoveEqual.ll
3,520 msLLVM.Transforms/MergeFunc::Unknown Unit Message ("")
Script: -- : 'RUN: at line 1'; /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/build/bin/opt -S -mergefunc -mergefunc-use-aliases < /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/llvm/test/Transforms/MergeFunc/alias.ll | /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/build/bin/FileCheck /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/llvm/test/Transforms/MergeFunc/alias.ll
3,610 msLLVM.Transforms/MergeFunc::Unknown Unit Message ("")
Script: -- : 'RUN: at line 1'; /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/build/bin/opt -S -mergefunc < /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/llvm/test/Transforms/MergeFunc/apply_function_attributes.ll | /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/build/bin/FileCheck /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/llvm/test/Transforms/MergeFunc/apply_function_attributes.ll
3,530 msLLVM.Transforms/MergeFunc::Unknown Unit Message ("")
Script: -- : 'RUN: at line 1'; /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/build/bin/opt -mergefunc -S < /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/llvm/test/Transforms/MergeFunc/call-and-invoke-with-ranges.ll | /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/build/bin/FileCheck /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/llvm/test/Transforms/MergeFunc/call-and-invoke-with-ranges.ll
3,250 msLLVM.Transforms/MergeFunc::Unknown Unit Message ("")
Script: -- : 'RUN: at line 1'; /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/build/bin/opt -S -mergefunc < /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/llvm/test/Transforms/MergeFunc/constant-entire-value.ll | /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/build/bin/FileCheck /mnt/disks/ssd0/agent/workspace/amd64_debian_testing_clang/llvm-project/llvm/test/Transforms/MergeFunc/constant-entire-value.ll
View Full Test Results (38 Failed)

Event Timeline

vish99 created this revision.Jun 7 2020, 12:22 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 7 2020, 12:22 PM
vish99 edited the summary of this revision. (Show Details)Jun 7 2020, 12:24 PM