Difference between revisions of "DIP39"

From D Wiki
Jump to: navigation, search
(Alternative symbols for call site rvalue annotation)
 
(25 intermediate revisions by the same user not shown)
Line 1: Line 1:
== DIP 39: Safe rvalue references: backwards compatible, safe against ref/nonref code evolution, compatible with UFCS and DIP38 ==
+
== DIP 39: Safe rvalue references: backwards compatible, escapable, safe against ref/nonref code evolution, compatible with UFCS and DIP38 ==
  
 
{| class="wikitable"
 
{| class="wikitable"
 
!Title:
 
!Title:
!''Safe rvalue references: backwards compatible, safe against ref/nonref code evolution, compatible with UFCS and DIP38.''
+
!''Safe rvalue references: backwards compatible, escapable, safe against ref/nonref code evolution, compatible with UFCS and DIP38.''
 
|-
 
|-
 
|DIP:
 
|DIP:
Line 9: Line 9:
 
|-
 
|-
 
|Version:
 
|Version:
|1
+
|2
 
|-
 
|-
 
|Status:
 
|Status:
|Draft
+
|
 
|-
 
|-
 
|Created:
 
|Created:
Line 18: Line 18:
 
|-
 
|-
 
|Last Modified:
 
|Last Modified:
|2013-05-10
+
|2013-05-12
 
|-
 
|-
 
|Author:
 
|Author:
Line 29: Line 29:
 
== Abstract ==
 
== Abstract ==
  
We propose to introduce rvalue references that are:
+
We propose to introduce rvalue (RV) references via a special call site annotation '''Expr^''' that creates a temporary from an expression Expr.
* safe: guarantees memory safety so that references will always point to valid memory.
+
 
* compatible with DIP38: can use same inref/outref internal compiler annotation for input references that can be returned by ref by a function.
+
Features:
* backwards compatible: current valid D code will continue to work without change. In addition, additional code becomes valid with call site rvalue ref annotation.
+
* backwards compatible: current valid D code will continue to work without change. In addition, RV references are made possible with a call site annotation
* safe against ref/nonref code evolution: call site rvalue ref compulsory annotation turns ref/nonref changes into compile errors instead of silently changing code behavior.
+
* does not introduce any change in function signature as in DIP36
 +
* allows to safely escape the RV reference, as opposed to DIP36 which didn't allow escaping those
 
* both const ref or ref can be used with rvalue refs (more flexible than C++)
 
* both const ref or ref can be used with rvalue refs (more flexible than C++)
* no call site ref annotation when input ref argument is already an lvalue (different from C#), for backwards compatibility (and making it less verbose)
+
* safe against ref/nonref code evolution: code that would change behavior due to ref/nonref signature change would result in compile errors rather than silently modify behavior
* compatible with UFCS
+
* can be used with UFCS
 +
* compatible with DIP38: can use same inref/outref internal compiler annotation (or explicit user annotation) for input references that can be returned by ref by a function.
 +
* safe: guarantees memory safety so that references will always point to valid memory (either runtime safety as proposed in Dconf13 or compile time safety through DIP38)
 +
 
 +
== Motivation ==
 +
Rvalue references is one of the rare features that C++ has over D. It allows to have both efficiency (passing by reference) and convenience (do not break UFCS pipelines for example).
 +
A recent DIP was proposed (DIP36) to support them but had the following shortcomings: it prevented returning such references from a function (useful in functions such as tap), and it required modifying the signature of the function with scope ref.
 +
This proposal addresses the shortcomings of DIP36, and works by requiring a call site annotation for converting rvalues to a temporary lvalue reference.
 +
Furthermore, given DIP38 (safety of refs) and DIP39 (rvalue refs), I would argue that pass by ref should be the new prefered way to take inputs in most cases.
  
 
== Details ==
 
== Details ==
Suppose we have a function that takes an input by ref:
+
Let us introduce some notation:
 +
* '''LV''' : denotes an lvalue expression
 +
* '''RV''' denotes an rvalue expression
 +
* '''Expr''' denotes an expression (could be an LV or RV)
 +
 
 +
We introduce the following new notation:
 +
* '''fun(Expr^)''' : creates a temporary variable from '''Expr''' before passing it to a function that takes '''Expr''' by ref.
 +
I propose the symbol '^' to denote this temporary creation (^ is also used for XOR in D but shouldn't create ambiguity as XOR is binary), although there are alternatives, see section: 'Alternative symbols for temporary creation'.
 +
 
 
<syntaxhighlight lang="d">
 
<syntaxhighlight lang="d">
 +
// Suppose we have a function:
 
T2 funRef(ref T a);
 
T2 funRef(ref T a);
</syntaxhighlight>
+
// We can use it as before with an LV (backwards compatible):
We can use it as before with an lvalue LV (backwards compatible):
 
<syntaxhighlight lang="d">
 
 
funRef(LV);  
 
funRef(LV);  
</syntaxhighlight>
+
// Our proposed new syntax also allows to call fun by creating a temporary from an expression:
We can use it with an rvalue expression 'RV' and a compulsory call site annotation indicating to convert the rvalue to an lvalue via a temporary:
+
funRef(Expr^);//create a temporary before calling funRef: 'auto _tmp=Expr; funRef(_tmp);'
 
 
I propose the symbol '^' to denote this (^ is also used for XOR but shouldn't create ambiguity), although there are alternatives, see section: 'alternative symbols for call site rvalue annotation'.
 
 
 
<syntaxhighlight lang="d">
 
funRef(RV^);
 
 
</syntaxhighlight>
 
</syntaxhighlight>
  
The rule is simple: 'fun(x^)' is used if and only if fun takes x by ref and x is an rvalue (not an lvalue). Otherwise use fun(x).
+
The rule is simple: funRef(ref T a) can still only take an LV, and 'Expr^' or '(Expr)^' creates a temporary LV from an Expr.  
with funRef(ref T a):
+
With funRef(ref T a):
* LV.funRef(); //ok
+
* funRef(LV); //ok: LV passed by ref
* RV^.funRef(); //ok
+
* funRef(RV); //error
* LV^.funRef(); //error
+
* funRef(Expr^); // ok: 'auto _tmp=Expr; funRef(_tmp);'
* RV.funRef(); //error
 
  
 
with funNonRef(T a):
 
with funNonRef(T a):
* LV.funNonRef(); //ok
+
* funNonRef(Expr); // ok
* RV.funNonRef(); //ok
+
* funNonRef(Expr^); // error
* LV^.funNonRef(); //error
 
* RV^.funNonRef(); //error
 
  
If passing an lvalue to a ref-taking function that involves an implicit conversion, then binding is disallowed without the call-site annotation (the LRL (Lvalue-Rvalue-Lvalue) problem pointed out by Andrei):
+
If passing an LV to a ref-taking function that involves an implicit conversion, then binding is disallowed without the call-site annotation; this is the LRL (Lvalue-Rvalue-Lvalue) problem pointed out by Andrei:
 
<syntaxhighlight lang="d">
 
<syntaxhighlight lang="d">
 
void fix(ref double x) { if (isnan(x)) x = 0; }
 
void fix(ref double x) { if (isnan(x)) x = 0; }
 
float a;
 
float a;
fix(a); // compilation error due to mismatched types
+
fix(a); // error due to mismatched types
 
</syntaxhighlight>
 
</syntaxhighlight>
 +
 +
With those rules, current valid code will stay valid, and new code becomes possible in a safe way.
  
 
== Implementation details ==
 
== Implementation details ==
The compiler will create a temporary whose lifetime shall survive the entire expression where this RV^ occurs:
+
The compiler will create a temporary whose lifetime shall survive the entire expression where '''Expr^''' occurs:
 
<syntaxhighlight lang="d">
 
<syntaxhighlight lang="d">
expr ( funRef(RV^)  )
+
expr ( funRef(Expr^)  )
 
//rewritten by compiler as:
 
//rewritten by compiler as:
auto _tmp=RV;
+
auto _tmp=Expr;
 
expr ( funRef(_tmp)  );
 
expr ( funRef(_tmp)  );
 
</syntaxhighlight>
 
</syntaxhighlight>
Line 89: Line 99:
 
with 'funRef(ref T a)':
 
with 'funRef(ref T a)':
 
* LV.funRef(); //ok
 
* LV.funRef(); //ok
* RV^.funRef(); //ok
 
* LV^.funRef(); //error
 
 
* RV.funRef(); //error
 
* RV.funRef(); //error
 +
* Expr^.funRef(); //ok
  
 
with 'funNonRef(T a)':
 
with 'funNonRef(T a)':
* LV.funNonRef(); //ok
+
* Expr.funNonRef(); //ok
* RV.funNonRef(); //ok
+
* Expr^.funNonRef(); //error
* LV^.funNonRef(); //error
 
* RV^.funNonRef(); //error
 
 
 
So current valid code will stay valid, and new code becomes possible in a safe way.*
 
  
 
== Safety against return by ref/nonref code evolution ==
 
== Safety against return by ref/nonref code evolution ==
 
+
As Andrei pointed out recently, introducing RV references must take into account the dangers of ref/nonref code evolution. Our solution is safe in this aspect:
This problem was pointed out by Andrei.
 
 
<syntaxhighlight lang="d">
 
<syntaxhighlight lang="d">
 
struct A{ref T opIndex(int i){...}}
 
struct A{ref T opIndex(int i){...}}
void fix(ref T x){if (isnan(x)) x = 0; }
+
void funRef(ref T x){if (isnan(x)) x = 0; }
 
A a;
 
A a;
fix(a[0]);//ok since a[0] is an lvalue.
+
funRef(a[0]);//ok since a[0] is an LV.
//later on the code changes: opIndex returns by value
+
//later on the code changes: opIndex returns by value 'T opIndex(int i)'.
fix(a[0]);//now becomes an error since a[0] becomes an rvalue! so we're prevented from accepting code that silently does the wrong thing.
+
funRef(a[0]);//now becomes an error since a[0] becomes an RV! so we're prevented from accepting code that silently does the wrong thing.
 +
</syntaxhighlight>
 +
 
 +
Let us consider the reverse evolution from nonref to ref:
 +
<syntaxhighlight lang="d">
 +
struct A{ T opIndex(int i){...}}
 +
void funRef(ref T x){...} // some function which may modify x
 +
funRef(a[0]^); //ok, a temporary is created from the RV a[0].
 +
// later on, code changes: opIndex returns by ref 'ref T opIndex(int i)'.
 +
funRef(a[0]^); //still ok, a temporary copy of a[0] (which is now an LV) is still created.
 
</syntaxhighlight>
 
</syntaxhighlight>
 +
In this second example, the code behavior remains unchanged before and after the opIndex signature change, as in both cases a temporary copy is created. This is especially relevant if funRef modifies its input in any way.
 +
Note: thanks to Dmitry S for proposing to accept LV^, motivated by this example (the original proposal only accepted RV^).
  
== Also: Safety against another type of code evolution: proposal to error on ignored nonref return value ==
+
== Safety against another type of code evolution ==
This proposal is independant of this DIP39 as it is independent of rvalue refs but I mention it here.
+
This section could be taken out of this DIP39 without affecting it as it is independent of RV refs, but it further enhances language safety against ref/nonref code evolution:
Actually another problem that has nothing to do with rvalue refs can also be addressed:
 
 
<syntaxhighlight lang="d">
 
<syntaxhighlight lang="d">
//suppose now the 'fix' code changes to :
+
void fix(ref T x){if (isnan(x)) x = 0; return x;}
 +
T x;
 +
fix(x); //ok
 +
// later, fix takes by nonref and returns by nonref:
 
T fix(T x){if (isnan(x)) x = 0; return x;}
 
T fix(T x){if (isnan(x)) x = 0; return x;}
fix(a[0]); //compiles but does nothing, regardless of opIndex returning value or ref.
+
fix(x); //compiles but does nothing!
 
</syntaxhighlight>
 
</syntaxhighlight>
  
Line 126: Line 143:
 
void ignore(T)(T a){}
 
void ignore(T)(T a){}
 
// can be used as:
 
// can be used as:
fix(a[0]); // error: nonref return value of array is ignored
+
fix(x); // error: nonref return value returned by fix(x) is ignored
fix(a[0]).ignore; // ok (although not very useful here; but would be for ignoring, say, error codes)
+
fix(x).ignore; // ok (although not very useful here; but would be for ignoring, say, printf C style error codes)
 
</syntaxhighlight>
 
</syntaxhighlight>
That'll address this issue.
 
This can be enabled with a compiler flag -error_ignored_nonref_return.
 
  
Note, ref returned values are safer to ignore as an object still persists on the stack, but we could also choose to error on those.
+
* Note1 :This can be enabled with a compiler flag -error_ignored_nonref_return.
 +
* Note2 :This will be compiled away so has no runtime performance penalty.
 +
* Note3: ref returned values are safer to ignore as ref returns should point to outside memory (either static object or temporary created by RV^ or an lvalue on the stack). Example: ref T (ref T x){if (isnan(x)) x = 0; return x;).
 +
We could also choose to error on all ignored returned values but that might break more code.
  
 
== Safety ==  
 
== Safety ==  
 
Memory safety would be the same as the current situation in D with same existing pitfalls and no new pitfalls introduced. In conjunction with DIP 38, memory safety would be guaranteed at compile time. With the one introduced in Dconf13, it would be guaranteed with a runtime check.
 
Memory safety would be the same as the current situation in D with same existing pitfalls and no new pitfalls introduced. In conjunction with DIP 38, memory safety would be guaranteed at compile time. With the one introduced in Dconf13, it would be guaranteed with a runtime check.
  
== Alternative symbols for call site rvalue annotation ==
+
== Alternative symbols for temporary cration ==
  
 
2 things to decide on : prefix or postfix annotation, and which annotation to use:  
 
2 things to decide on : prefix or postfix annotation, and which annotation to use:  
Line 172: Line 190:
 
This document has been placed in the Public Domain.
 
This document has been placed in the Public Domain.
  
Thanks to Dmitry S for corrections.
+
Thanks to Dmitry S for useful suggestions.
 
[[Category: DIP]]
 
[[Category: DIP]]

Latest revision as of 01:18, 13 May 2013

DIP 39: Safe rvalue references: backwards compatible, escapable, safe against ref/nonref code evolution, compatible with UFCS and DIP38

Title: Safe rvalue references: backwards compatible, escapable, safe against ref/nonref code evolution, compatible with UFCS and DIP38.
DIP: 39
Version: 2
Status:
Created: 2013-05-10
Last Modified: 2013-05-12
Author: Timothee Cour
Links:

Abstract

We propose to introduce rvalue (RV) references via a special call site annotation Expr^ that creates a temporary from an expression Expr.

Features:

  • backwards compatible: current valid D code will continue to work without change. In addition, RV references are made possible with a call site annotation
  • does not introduce any change in function signature as in DIP36
  • allows to safely escape the RV reference, as opposed to DIP36 which didn't allow escaping those
  • both const ref or ref can be used with rvalue refs (more flexible than C++)
  • safe against ref/nonref code evolution: code that would change behavior due to ref/nonref signature change would result in compile errors rather than silently modify behavior
  • can be used with UFCS
  • compatible with DIP38: can use same inref/outref internal compiler annotation (or explicit user annotation) for input references that can be returned by ref by a function.
  • safe: guarantees memory safety so that references will always point to valid memory (either runtime safety as proposed in Dconf13 or compile time safety through DIP38)

Motivation

Rvalue references is one of the rare features that C++ has over D. It allows to have both efficiency (passing by reference) and convenience (do not break UFCS pipelines for example). A recent DIP was proposed (DIP36) to support them but had the following shortcomings: it prevented returning such references from a function (useful in functions such as tap), and it required modifying the signature of the function with scope ref. This proposal addresses the shortcomings of DIP36, and works by requiring a call site annotation for converting rvalues to a temporary lvalue reference. Furthermore, given DIP38 (safety of refs) and DIP39 (rvalue refs), I would argue that pass by ref should be the new prefered way to take inputs in most cases.

Details

Let us introduce some notation:

  • LV : denotes an lvalue expression
  • RV denotes an rvalue expression
  • Expr denotes an expression (could be an LV or RV)

We introduce the following new notation:

  • fun(Expr^) : creates a temporary variable from Expr before passing it to a function that takes Expr by ref.

I propose the symbol '^' to denote this temporary creation (^ is also used for XOR in D but shouldn't create ambiguity as XOR is binary), although there are alternatives, see section: 'Alternative symbols for temporary creation'.

// Suppose we have a function:
T2 funRef(ref T a);
// We can use it as before with an LV (backwards compatible):
funRef(LV); 
// Our proposed new syntax also allows to call fun by creating a temporary from an expression:
funRef(Expr^);//create a temporary before calling funRef: 'auto _tmp=Expr; funRef(_tmp);'

The rule is simple: funRef(ref T a) can still only take an LV, and 'Expr^' or '(Expr)^' creates a temporary LV from an Expr. With funRef(ref T a):

  • funRef(LV); //ok: LV passed by ref
  • funRef(RV); //error
  • funRef(Expr^); // ok: 'auto _tmp=Expr; funRef(_tmp);'

with funNonRef(T a):

  • funNonRef(Expr); // ok
  • funNonRef(Expr^); // error

If passing an LV to a ref-taking function that involves an implicit conversion, then binding is disallowed without the call-site annotation; this is the LRL (Lvalue-Rvalue-Lvalue) problem pointed out by Andrei:

void fix(ref double x) { if (isnan(x)) x = 0; }
float a;
fix(a); // error due to mismatched types

With those rules, current valid code will stay valid, and new code becomes possible in a safe way.

Implementation details

The compiler will create a temporary whose lifetime shall survive the entire expression where Expr^ occurs:

expr ( funRef(Expr^)  )
//rewritten by compiler as:
auto _tmp=Expr;
expr ( funRef(_tmp)  );

UFCS

The rule for UFCS is the same:

with 'funRef(ref T a)':

  • LV.funRef(); //ok
  • RV.funRef(); //error
  • Expr^.funRef(); //ok

with 'funNonRef(T a)':

  • Expr.funNonRef(); //ok
  • Expr^.funNonRef(); //error

Safety against return by ref/nonref code evolution

As Andrei pointed out recently, introducing RV references must take into account the dangers of ref/nonref code evolution. Our solution is safe in this aspect:

struct A{ref T opIndex(int i){...}}
void funRef(ref T x){if (isnan(x)) x = 0; }
A a;
funRef(a[0]);//ok since a[0] is an LV.
//later on the code changes: opIndex returns by value 'T opIndex(int i)'.
funRef(a[0]);//now becomes an error since a[0] becomes an RV! so we're prevented from accepting code that silently does the wrong thing.

Let us consider the reverse evolution from nonref to ref:

struct A{ T opIndex(int i){...}}
void funRef(ref T x){...} // some function which may modify x
funRef(a[0]^); //ok, a temporary is created from the RV a[0]. 
// later on, code changes: opIndex returns by ref 'ref T opIndex(int i)'.
funRef(a[0]^); //still ok, a temporary copy of a[0] (which is now an LV) is still created.

In this second example, the code behavior remains unchanged before and after the opIndex signature change, as in both cases a temporary copy is created. This is especially relevant if funRef modifies its input in any way. Note: thanks to Dmitry S for proposing to accept LV^, motivated by this example (the original proposal only accepted RV^).

Safety against another type of code evolution

This section could be taken out of this DIP39 without affecting it as it is independent of RV refs, but it further enhances language safety against ref/nonref code evolution:

void fix(ref T x){if (isnan(x)) x = 0; return x;}
T x;
fix(x); //ok
// later, fix takes by nonref and returns by nonref:
T fix(T x){if (isnan(x)) x = 0; return x;}
fix(x); //compiles but does nothing!

I propose to make it an error to ignore a nonref return value, and to add a function 'ignore' for convenience that consumes and does nothing:

void ignore(T)(T a){}
// can be used as:
fix(x); // error: nonref return value returned by fix(x) is ignored
fix(x).ignore; // ok (although not very useful here; but would be for ignoring, say, printf C style error codes)
  • Note1 :This can be enabled with a compiler flag -error_ignored_nonref_return.
  • Note2 :This will be compiled away so has no runtime performance penalty.
  • Note3: ref returned values are safer to ignore as ref returns should point to outside memory (either static object or temporary created by RV^ or an lvalue on the stack). Example: ref T (ref T x){if (isnan(x)) x = 0; return x;).

We could also choose to error on all ignored returned values but that might break more code.

Safety

Memory safety would be the same as the current situation in D with same existing pitfalls and no new pitfalls introduced. In conjunction with DIP 38, memory safety would be guaranteed at compile time. With the one introduced in Dconf13, it would be guaranteed with a runtime check.

Alternative symbols for temporary cration

2 things to decide on : prefix or postfix annotation, and which annotation to use:

prefix vs postfix:

  • postfix fun(RV^): (proposed): compatible with left-to-right pipelines in D: [1,2].sort.map!fun.uniq
  • prefix fun(^RV): compatible with '&' location wrt RV argument

This can affect ease of disambiguation wrt existing symbols.

which annotation to use (regardless of prefix/postfix):

  • fun(RV^);//(proposed). Used for XOR but should not be ambiguous; reminds of a C++ special reference extension
  • fun(RV@);//@ has UDA meaning in D, but that could be made unambiguous
  • fun(auto(RV)); // suggested by Dmitry S; reminds of creating a temporary variable with auto-deduced type, analog to int(x + 1)
  • fun(RV.auto); // UFCS version of auto(RV), makes UFCS much nicer
  • likewise with ref(RV) and RV.ref //reminds of C# call site annotation via 'ref', and reminds of function signature
  • fun(RV#); //# has a special line reordering meaning in D, but that could be made unambiguous
  • fun(RV?); //? has a special (a?b:c) meaning in D, but that could be made unambiguous
  • fun(RV&); //probably a bad idea, since for a templated function fun(T)(ref Ta ) this could call fun!(typeof(RV*))(RV&)

This could look like this:

T fun1(ref T2);
T fun2(ref T2);
//with RV^:
[1,2]^.fun1^.fun2.writeln;
//with auto(RV):
auto(auto([1,2]).fun1).fun2.writeln; // a bit verbose
//with RV.auto:
[1,2].auto.fun1.auto.fun2.writeln; // better

Copyright

This document has been placed in the Public Domain.

Thanks to Dmitry S for useful suggestions.