[dmd-internals] [D-Programming-Language/dmd] 95367e: Split mangle test into two files

GitHub noreply at github.com
Tue Feb 18 09:25:23 PST 2014


  Branch: refs/heads/master
  Home:   https://github.com/D-Programming-Language/dmd
  Commit: 95367ee1577c20ca74aa032bc0aae2fab8f12946
      https://github.com/D-Programming-Language/dmd/commit/95367ee1577c20ca74aa032bc0aae2fab8f12946
  Author: Johannes Pfau <johannespfau at gmail.com>
  Date:   2014-02-18 (Tue, 18 Feb 2014)

  Changed paths:
    A test/runnable/imports/mangle1.d
    M test/runnable/mangle.d

  Log Message:
  -----------
  Split mangle test into two files

We currently have two VarDeclarations in mangle.d which reference
the same symbol. This causes problems in the GCC backend as it does
not realize it actually deals with the same symbol and therefore
reorders reads and writes. It's not clear if this will be fixed in the
GCC backend at all as the fix would break other stuff. It's also
kind of weird to reference a symbol in this way. The correct way
is to use aliases.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60214

Anyway, this test was likely not supposed to test if the compiler
recognizes that these are the same variables, it was probably just to
test that we can read/write to a symbol with a specific mangle.


  Commit: cfed38e2bfa0f3956247aaff986e653991846cfa
      https://github.com/D-Programming-Language/dmd/commit/cfed38e2bfa0f3956247aaff986e653991846cfa
  Author: Daniel Murphy <yebblies at gmail.com>
  Date:   2014-02-18 (Tue, 18 Feb 2014)

  Changed paths:
    A test/runnable/imports/mangle1.d
    M test/runnable/mangle.d

  Log Message:
  -----------
  Merge pull request #3280 from jpf91/mangle

Split mangle test into two files


Compare: https://github.com/D-Programming-Language/dmd/compare/7e5347bc6007...cfed38e2bfa0


More information about the dmd-internals mailing list