[parrot/parrot] 012a78: [imcc] optimize used_once - cage

GitHub noreply at github.com
Thu Nov 13 10:58:54 UTC 2014


  Branch: refs/heads/smoke-me/ops2c-parens-gh1008
  Home:   https://github.com/parrot/parrot
  Commit: 012a78b0be1832b453fd1bca029e6633f4e6cd2e
      https://github.com/parrot/parrot/commit/012a78b0be1832b453fd1bca029e6633f4e6cd2e
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2014-11-13 (Thu, 13 Nov 2014)

  Changed paths:
    M compilers/imcc/optimizer.c
    M compilers/opsc/src/Ops/Emitter.pm

  Log Message:
  -----------
  [imcc] optimize used_once - cage

address of array ins->symregs will always evaluate to true, statically allocated
check ins->symregs[0] instead.
Detected with clang-3.5++

Whitespace collapsing in opsc/src/Ops/Emitter


  Commit: 277c3386a4b6e65a6708c685889a4c6fbbcf35ff
      https://github.com/parrot/parrot/commit/277c3386a4b6e65a6708c685889a4c6fbbcf35ff
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2014-11-13 (Thu, 13 Nov 2014)

  Changed paths:
    M compilers/opsc/ops2c.nqp
    M compilers/opsc/src/Ops/Op.pm
    M src/ops/core_ops.c

  Log Message:
  -----------
  [ops2c]: remove extraneous parentheses

no need for -Wno-parentheses-equality anymore.
Closes #1008


  Commit: dc45c0cb528f12a8125bc893fe985e2cc0723fb7
      https://github.com/parrot/parrot/commit/dc45c0cb528f12a8125bc893fe985e2cc0723fb7
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2014-11-13 (Thu, 13 Nov 2014)

  Changed paths:
    M compilers/imcc/Rules.in
    M config/auto/warnings.pm
    M config/gen/makefiles/root.in
    M config/init/optimize.pm
    M src/dynoplibs/Rules.in

  Log Message:
  -----------
  [cage] remove -Wno-parentheses-equality, SUFFIX OVERRIDE

Remove unneeded SUFFIX OVERRIDE for dynoplibs
cc_shared is not part of CFLAGS, so remove those special cases also
add -Wno-switch-default to generated compilers/imcc/imcparser.c

Tested with gcc49, clang3.5 and clang3.5++


  Commit: 2b3d00a26bb903356269147134fcd33592348f1b
      https://github.com/parrot/parrot/commit/2b3d00a26bb903356269147134fcd33592348f1b
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2014-11-13 (Thu, 13 Nov 2014)

  Changed paths:
    M ChangeLog
    M config/auto/warnings.pm
    M config/gen/makefiles/root.in
    M include/parrot/namespace.h
    M src/dynoplibs/Rules.in
    M src/namespace.c

  Log Message:
  -----------
  [cage] adjust all warnings and overrides

no wrong warnings in parrot core anymore
tested: gcc4.4, gcc4.5, gcc4.6, gcc4.8, gcc4.9,
  clang3.5, clang3.5++, apple cc

remaining todo:
* --optimize -Wunused-parameter: interp used with ifndef NDEBUG
* -Wcast-qual with gcc, but these warnings are legit.
* stack usage: legit


  Commit: 73434879ec05fedefb6f41edfd41e5d902d993bd
      https://github.com/parrot/parrot/commit/73434879ec05fedefb6f41edfd41e5d902d993bd
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2014-11-13 (Thu, 13 Nov 2014)

  Changed paths:
    M src/string/spf_render.c

  Log Message:
  -----------
  [cage] spf_render.c UINTVAL_FMT: no %d format for UINTVAL


  Commit: 17726aed74daed448cd576d066c2bf69d85d747c
      https://github.com/parrot/parrot/commit/17726aed74daed448cd576d066c2bf69d85d747c
  Author: Reini Urban <rurban at cpanel.net>
  Date:   2014-11-13 (Thu, 13 Nov 2014)

  Changed paths:
    M src/call/context.c

  Log Message:
  -----------
  [cage] fix warn_unused_result with Parrot_set_new_context


Compare: https://github.com/parrot/parrot/compare/012a78b0be18^...17726aed74da


More information about the parrot-commits mailing list