Differential D123125

[Attributor] Update min-legal-vector-width when do promotion
AbandonedPublic
Actions

Authored by pengfei on Apr 5 2022, 6:01 AM.

Tags

Subscribers

Details

Reviewers

efriedma
jdoerfert
craig.topper
uenoku
homerdin
sstefan1
baziotis

Diff Detail

Repository: rG LLVM Github Monorepo

Unit TestsFailed
View All

	Time	Test
	40 ms	x64 debian > ScudoStandalone-Unit-GwpAsanTorture._/ScudoCUnitTest-x86_64-Test::ScudoWrappersCTest.MallocInfo Script: -- /var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/lib/scudo/standalone/tests/./ScudoCUnitTest-x86_64-Test --gtest_filter=ScudoWrappersCTest.MallocInfo
	20 ms	x64 debian > ScudoStandalone-Unit._/ScudoCUnitTest-x86_64-Test::ScudoWrappersCTest.MallocInfo Script: -- /var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/lib/scudo/standalone/tests/./ScudoCUnitTest-x86_64-Test --gtest_filter=ScudoWrappersCTest.MallocInfo
	60,070 ms	x64 debian > libFuzzer.libFuzzer::large.test Script: -- : 'RUN: at line 3'; /var/lib/buildkite-agent/builds/llvm-project/build/./bin/clang --driver-mode=g++ -O2 -gline-tables-only -fsanitize=address,fuzzer -I/var/lib/buildkite-agent/builds/llvm-project/compiler-rt/lib/fuzzer -m64 /var/lib/buildkite-agent/builds/llvm-project/compiler-rt/test/fuzzer/LargeTest.cpp -o /var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/test/fuzzer/X86_64DefaultLinuxConfig/Output/large.test.tmp-LargeTest

Event Timeline

pengfei created this revision.Apr 5 2022, 6:01 AM

Herald added a reviewer: homerdin. · View Herald TranscriptApr 5 2022, 6:01 AM

Herald added a project: Restricted Project. · View Herald Transcript

Herald added subscribers: ormris, StephenFan, okura and 2 others. · View Herald Transcript

pengfei requested review of this revision.Apr 5 2022, 6:01 AM

Herald added a reviewer: sstefan1. · View Herald TranscriptApr 5 2022, 6:01 AM

Herald added a reviewer: baziotis. · View Herald Transcript

Herald added a project: Restricted Project. · View Herald Transcript

Herald added a subscriber: llvm-commits. · View Herald Transcript

pengfei mentioned this in D122594: [ArgPromotion] Update min-legal-vector-width when do promotion.Apr 5 2022, 6:15 AM

pengfei added a child revision: D123124: [Verifier] Add verifier for min-legal-vector-width.

Harbormaster completed remote builds in B157959: Diff 420478.Apr 5 2022, 7:28 AM

Thanks for the patch!

Could you add a commit message explaining the motivation and what this does.
I also left a question below.

llvm/lib/Transforms/IPO/Attributor.cpp
2498	Should this be a helper, seems reausable, no?

pengfei mentioned this in D123284: [ArgPromotion][Attributor] Update min-legal-vector-width when do promotion.Apr 7 2022, 12:10 AM

pengfei added inline comments.Apr 7 2022, 12:36 AM

llvm/lib/Transforms/IPO/Attributor.cpp
2498	Good point! Added a helper and merged D122594 together to D123284 .

pengfei abandoned this revision.Apr 7 2022, 12:40 AM

pengfei removed a child revision: D123124: [Verifier] Add verifier for min-legal-vector-width.

Revision Contents
Changeset List

Path

Size

llvm/

lib/

Transforms/

IPO/

26 lines

test/

Transforms/

Attributor/

ArgumentPromotion/

X86/

min-legal-vector-width.ll

118 lines

Diff 420478

llvm/lib/Transforms/IPO/Attributor.cpp

Loading...

llvm/test/Transforms/Attributor/ArgumentPromotion/X86/min-legal-vector-width.ll

Loading...