HomePhabricator

[analyzer] Don't delete TaintConfig copy constructor

Description

[analyzer] Don't delete TaintConfig copy constructor

Summary:
Explicitly deleting the copy constructor makes compiling the function
ento::registerGenericTaintChecker difficult with some compilers. When we
construct an llvm::Optional<TaintConfig>, the optional is constructed with a
const TaintConfig reference which it then uses to invoke the deleted TaintConfig
copy constructor.

I've observered this failing with clang 3.8 on Ubuntu 16.04.

Reviewers: compnerd, Szelethus, boga95, NoQ, alexshap

Subscribers: xazax.hun, baloghadamsoftware, szepet, a.sidorin, mikhail.ramalho, donat.nagy, dkrupp, Charusso, llvm-commits, cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D66192

Details

Committed
xiaobaiAug 13 2019, 6:09 PM
Differential Revision
D66192: [analyzer] Don't delete TaintConfig copy constructor
Parents
rL368778: [ORC] Fix clang-interpreter example code broken by r368707.
Branches
Unknown
Tags
Unknown