Page MenuHomePhabricator

[VE] Support atomic load
AbandonedPublic

Authored by kaz7 on Oct 24 2020, 12:33 AM.

Details

Reviewers
simoll
k-ishizaka
Summary

Support atomic load instruction and add a regression test.

Diff Detail

Unit TestsFailed

TimeTest
410 mslinux > HWAddressSanitizer-x86_64.TestCases::sizes.cpp
Script: -- : 'RUN: at line 3'; /mnt/disks/ssd0/agent/llvm-project/build/./bin/clang --driver-mode=g++ -m64 -gline-tables-only -fsanitize=hwaddress -fuse-ld=lld -mcmodel=large -mllvm -hwasan-globals -mllvm -hwasan-use-short-granules -mllvm -hwasan-instrument-landing-pads=0 -mllvm -hwasan-instrument-personality-functions /mnt/disks/ssd0/agent/llvm-project/compiler-rt/test/hwasan/TestCases/sizes.cpp -nostdlib++ -lstdc++ -o /mnt/disks/ssd0/agent/llvm-project/build/projects/compiler-rt/test/hwasan/X86_64/TestCases/Output/sizes.cpp.tmp

Event Timeline

kaz7 created this revision.Oct 24 2020, 12:33 AM
kaz7 requested review of this revision.Oct 24 2020, 12:33 AM
kaz7 updated this revision to Diff 300467.Oct 24 2020, 2:12 AM

Update a regression test.

kaz7 updated this revision to Diff 300468.Oct 24 2020, 2:27 AM

Update commit message

kaz7 retitled this revision from [VE] Support atomic_load to [VE] Support atomic load.Oct 24 2020, 2:28 AM
kaz7 edited the summary of this revision. (Show Details)
Harbormaster completed remote builds in B76278: Diff 300467.
kaz7 abandoned this revision.Oct 24 2020, 8:36 PM

It's much better if I PR fence instruction first and load later. I abandon this one and send fence first.