[llvm] r198803 - Fix the C++03 build.

dblaikie at gmail.com dblaikie at gmail.com
Thu Jan 9 10:59:30 PST 2014


On Wed Jan 08 2014 at 2:33:34 PM, Rafael Espindola <
rafael.espindola at gmail.com> wrote:

> Author: rafael
> Date: Wed Jan  8 16:27:04 2014
> New Revision: 198803
>
> URL: http://llvm.org/viewvc/llvm-project?rev=198803&view=rev
> Log:
> Fix the C++03 build.
>
> With c++11 we never instantiate the copy constructor.
>

Could we add a unit test for that?


>
> Modified:
>     llvm/trunk/include/llvm/Support/ErrorOr.h
>
> Modified: llvm/trunk/include/llvm/Support/ErrorOr.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/
> llvm/Support/ErrorOr.h?rev=198803&r1=198802&r2=198803&view=diff
> ============================================================
> ==================
> --- llvm/trunk/include/llvm/Support/ErrorOr.h (original)
> +++ llvm/trunk/include/llvm/Support/ErrorOr.h Wed Jan  8 16:27:04 2014
> @@ -203,7 +203,7 @@ private:
>      } else {
>        // Get other's error.
>        HasError = true;
> -      new (getErrorStorage()) error_code(Other);
> +      new (getErrorStorage()) error_code(Other.getError());
>      }
>    }
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140109/b88bda22/attachment.html>


More information about the llvm-commits mailing list