Thread overview
[Issue 13826] Move volatileLoad/Store to core.volatile when the volatile keyword is removed
Aug 28, 2016
Mike
Oct 25, 2018
Iain Buclaw
Aug 29, 2019
Dlang Bot
Aug 29, 2019
Dlang Bot
Aug 29, 2019
Dlang Bot
Aug 29, 2019
Dlang Bot
August 28, 2016
https://issues.dlang.org/show_bug.cgi?id=13826

--- Comment #1 from Mike <slavo5150@yahoo.com> ---
volatile keyword was removed in March 2016: https://github.com/dlang/dmd/pull/5556

--
October 25, 2018
https://issues.dlang.org/show_bug.cgi?id=13826

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ibuclaw@gdcproject.org

--- Comment #2 from Iain Buclaw <ibuclaw@gdcproject.org> ---
Compiler support: https://github.com/dlang/dmd/pull/8869

--
November 03, 2018
https://issues.dlang.org/show_bug.cgi?id=13826

--- Comment #3 from github-bugzilla@puremagic.com ---
Commit pushed to master at https://github.com/dlang/dmd

https://github.com/dlang/dmd/commit/3a29b26449d9a214800e122e1da200626fa66be0 Merge pull request #8869 from ibuclaw/issue13826

Add compiler support for core.volatile.volatileLoad/Store

--
August 29, 2019
https://issues.dlang.org/show_bug.cgi?id=13826

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> ---
@ErnyTech created dlang/druntime pull request #2773 "Fix Issue 13826 - Move volatileLoad/Store to core.volatile when the volatile keyword is removed" fixing this issue:

- Fix Issue 13826 - Move volatileLoad/Store to core.volatile when the volatile keyword is removed

  This PR move the intrinsic volatileLoad and volatileStore into core.volatile,
  as promised by the discussion in https://github.com/dlang/druntime/pull/892
  and https://github.com/dlang/dmd/pull/4155.

  Currently an alias is maintained in core.bitop to avoid broken code,
  the alias is marked as deprecated to warn users of the displacement
  occurred in the core.volatile

  You should probably schedule a removal of the alias from core.bitop and
  publish it in https://dlang.org/deprecate.html

  Signed-off-by: Ernesto Castellotti <erny.castell@gmail.com>

https://github.com/dlang/druntime/pull/2773

--
August 29, 2019
https://issues.dlang.org/show_bug.cgi?id=13826

--- Comment #5 from Dlang Bot <dlang-bot@dlang.rocks> ---
@ErnyTech created dlang/dmd pull request #10370 "Fix Issue 13826 - Move volatileLoad/Store to core.volatile when the volatile keyword is removed" fixing this issue:

- Fix Issue 13826 - Move volatileLoad/Store to core.volatile when the volatile keyword is removed

  This PR move the intrinsic volatileLoad and volatileStore into core.volatile,
  as promised by the discussion in #892
  and dlang/dmd#4155.

  This PR removes the support for the intrinsic of volatileLoad and
  volatileStore in the core.bitop module

  This PR depends on: https://github.com/dlang/druntime/pull/2773

  Signed-off-by: Ernesto Castellotti <erny.castell@gmail.com>

https://github.com/dlang/dmd/pull/10370

--
August 29, 2019
https://issues.dlang.org/show_bug.cgi?id=13826

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #6 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/druntime pull request #2773 "Fix Issue 13826 - Move volatileLoad/Store to core.volatile when the volatile keyword is removed" was merged into master:

- bc1ef748ca37533500f6d510bef854420ce6febf by Ernesto Castellotti:
  Fix Issue 13826 - Move volatileLoad/Store to core.volatile when the volatile
keyword is removed

  This PR move the intrinsic volatileLoad and volatileStore into core.volatile,
  as promised by the discussion in #892
  and dlang/dmd#4155.

  Currently an alias is maintained in core.bitop to avoid broken code,
  the alias is marked as deprecated to warn users of the displacement
  occurred in the core.volatile

  You should probably schedule a removal of the alias from core.bitop and
  publish it in https://dlang.org/deprecate.html

  Signed-off-by: Ernesto Castellotti <erny.castell@gmail.com>

https://github.com/dlang/druntime/pull/2773

--
August 29, 2019
https://issues.dlang.org/show_bug.cgi?id=13826

--- Comment #7 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #10370 "Fix Issue 13826 - Move volatileLoad/Store to core.volatile when the volatile keyword is removed" was merged into master:

- e0e2071e059d6e2849e1fc962d61f891c2d100f2 by Ernesto Castellotti:
  Fix Issue 13826 - Move volatileLoad/Store to core.volatile when the volatile
keyword is removed

  This PR move the intrinsic volatileLoad and volatileStore into core.volatile,
  as promised by the discussion in #892
  and #4155.

  This PR removes the support for the intrinsic of volatileLoad and
  volatileStore in the core.bitop module

  This PR depends on: dlang/druntime#2773

  Signed-off-by: Ernesto Castellotti <erny.castell@gmail.com>

https://github.com/dlang/dmd/pull/10370

--