This is an archive of the discontinued LLVM Phabricator instance.

Differential D106097

[analyzer] Improving test case for AnanlysisConsumer
Needs ReviewPublic

Authored by RithikSharma on Jul 15 2021, 1:39 PM.

Details

Reviewers
NoQ
xazax.hun
teemperor
vsavchenko
Summary

This patch is a part of an attempt to obtain more timer data from the analyzer. In this patch, we have added some improvements to the test cases.

Diff Detail

Unit TestsFailed

TimeTest
2,590 msx64 debian > libarcher.critical::critical.c
2,750 msx64 debian > libarcher.critical::lock.c
2,470 msx64 debian > libarcher.parallel::parallel-firstprivate.c
2,380 msx64 debian > libarcher.parallel::parallel-simple.c
2,500 msx64 debian > libarcher.parallel::parallel-simple2.c
View Full Test Results (20 Failed)

Event Timeline

RithikSharma created this revision.Jul 15 2021, 1:39 PM
Herald added subscribers: manas, steakhal, ASDenysPetrov and 8 others. · View Herald TranscriptJul 15 2021, 1:39 PM
RithikSharma requested review of this revision.Jul 15 2021, 1:39 PM
Harbormaster completed remote builds in B114343: Diff 359111.Jul 15 2021, 3:31 PM

Revision Contents

PathSize
clang/
test/
Analysis/
16 lines

Diff 359111

clang/test/Analysis/analyzer-display-progress.cpp

Show All 14 Lines
namespace ns { namespace ns {
struct SomeStruct { struct SomeStruct {
void f(int) {} void f(int) {}
void f(float, ::SomeStruct) {} void f(float, ::SomeStruct) {}
void f(float, SomeStruct) {} void f(float, SomeStruct) {}
}; };
} }
// CHECK: analyzer-display-progress.cpp f() : {{[0-9]+}} // CHECK: analyzer-display-progress.cpp f() : {{[0-9]+\.[0-9]+}} ms
// CHECK: analyzer-display-progress.cpp g() : {{[0-9]+}} // CHECK: analyzer-display-progress.cpp g() : {{[0-9]+\.[0-9]+}} ms
// CHECK: analyzer-display-progress.cpp h() : {{[0-9]+}} // CHECK: analyzer-display-progress.cpp h() : {{[0-9]+\.[0-9]+}} ms
// CHECK: analyzer-display-progress.cpp SomeStruct::f() : {{[0-9]+}} // CHECK: analyzer-display-progress.cpp SomeStruct::f() : {{[0-9]+\.[0-9]+}} ms
// CHECK: analyzer-display-progress.cpp SomeOtherStruct::f() : {{[0-9]+}} // CHECK: analyzer-display-progress.cpp SomeOtherStruct::f() : {{[0-9]+\.[0-9]+}} ms
// CHECK: analyzer-display-progress.cpp ns::SomeStruct::f(int) : {{[0-9]+}} // CHECK: analyzer-display-progress.cpp ns::SomeStruct::f(int) : {{[0-9]+\.[0-9]+}} ms
// CHECK: analyzer-display-progress.cpp ns::SomeStruct::f(float, ::SomeStruct) : {{[0-9]+}} // CHECK: analyzer-display-progress.cpp ns::SomeStruct::f(float, ::SomeStruct) : {{[0-9]+\.[0-9]+}} ms
// CHECK: analyzer-display-progress.cpp ns::SomeStruct::f(float, struct ns::SomeStruct) : {{[0-9]+}} // CHECK: analyzer-display-progress.cpp ns::SomeStruct::f(float, struct ns::SomeStruct) : {{[0-9]+\.[0-9]+}} ms