From f650c0afd4db97ec9a3f07c1b6a652caede8a6f5 Mon Sep 17 00:00:00 2001 From: Reid Kleckner Date: Thu, 7 Aug 2014 01:00:42 +0000 Subject: [PATCH] Make FullExprArg trivially copyable by removing the copy ctor The default copy ctor is good. This avoids using inalloca for a bunch of Sema methods when self-hosting. llvm-svn: 215047 --- clang/include/clang/Sema/Sema.h | 5 ----- 1 file changed, 5 deletions(-) diff --git a/clang/include/clang/Sema/Sema.h b/clang/include/clang/Sema/Sema.h index 87453a25b4f8..ed31a9f86e2f 100644 --- a/clang/include/clang/Sema/Sema.h +++ b/clang/include/clang/Sema/Sema.h @@ -2946,11 +2946,6 @@ public: public: FullExprArg(Sema &actions) : E(nullptr) { } - // FIXME: The const_cast here is ugly. RValue references would make this - // much nicer (or we could duplicate a bunch of the move semantics - // emulation code from Ownership.h). - FullExprArg(const FullExprArg& Other) : E(Other.E) {} - ExprResult release() { return E; }