[dylan-lang/opendylan] 8c01a8: [dfmc-c-back-end] UWP needs volatile locals.

GitHub <[email protected]>
Newsgroups gmane.comp.lang.dylan.gwydion.cvs
Message-ID <5582516c7957d_4b4f3fa6fa5ed2c0737f3@hookshot-fe1-cp1-prd.iad.github.net.mail>
  Branch: refs/heads/master
  Home:   https://github.com/dylan-lang/opendylan
  Commit: 8c01a876118b66ee424d3799a513a48d774dbcbc
      https://github.com/dylan-lang/opendylan/commit/8c01a876118b66ee424d3799a513a48d774dbcbc
  Author: Bruce Mitchener <[email protected]>
  Date:   2015-06-17 (Wed, 17 Jun 2015)

  Changed paths:
    M sources/dfmc/c-back-end/c-back-end.dylan
    M sources/dfmc/c-back-end/c-emit-c-object.dylan
    M sources/dfmc/c-back-end/c-emit-object.dylan

  Log Message:
  -----------
  [dfmc-c-back-end] UWP needs volatile locals.

Previously, we only made locals volatile when the function
body contained a block with a non-local entry state. They
actually should be volatile if there is any <unwind-protect>
involved as well.

* sources/dfmc/c-back-end/c-emit-object.dylan
  (emit-lambda-body-using-function): Factor out code and call
    need-volatile-locals?.

* sources/dfmc/c-back-end/c-emit-c-object.dylan
  (emit-lambda-body-using-function): Factor out code and call
    need-volatile-locals?.

* sources/dfmc/c-back-end/c-back-end.dylan
  (need-volatile-locals?): New function factored out from the
    above, but also checks for <unwind-protect>.


  Commit: afd759569b7440c69dfa8761eff32872dd543aad
      https://github.com/dylan-lang/opendylan/commit/afd759569b7440c69dfa8761eff32872dd543aad
  Author: Peter S. Housel <[email protected]>
  Date:   2015-06-17 (Wed, 17 Jun 2015)

  Changed paths:
    M sources/dfmc/c-back-end/c-back-end.dylan
    M sources/dfmc/c-back-end/c-emit-c-object.dylan
    M sources/dfmc/c-back-end/c-emit-object.dylan

  Log Message:
  -----------
  Merge pull request #926 from waywardmonkeys/volatile-with-unwind-protect

[dfmc-c-back-end] UWP needs volatile locals.


Compare: https://github.com/dylan-lang/opendylan/compare/163b7db9faf0...afd759569b74

_______________________________________________
chatter mailing list
[email protected]
https://lists.opendylan.org/mailman/listinfo/chatter
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.