MyDeveloperDay added a comment.

This is having an impact on code in flang, I don't deny it might be better, but 
it may not be what is wanted.

  -  explicit Expr(const Scalar<Result> &x) : u{Constant<Result>{x}} {}
  -  explicit Expr(Scalar<Result> &&x) : u{Constant<Result>{std::move(x)}} {}
  +  explicit Expr(const Scalar<Result> &x) : u {
  +    Constant<Result> { x }
  +  }
  +  {}
  +  explicit Expr(Scalar<Result> &&x) : u {
  +    Constant<Result> { std::move(x) }
  +  }
  +  {}


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114583/new/

https://reviews.llvm.org/D114583

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to