Fix assertion failure in global-merge with unused ConstantExpr
authorOliver Stannard <oliver.stannard@arm.com>
Mon, 8 Jun 2015 16:55:31 +0000 (16:55 +0000)
committerOliver Stannard <oliver.stannard@arm.com>
Mon, 8 Jun 2015 16:55:31 +0000 (16:55 +0000)
commit8379e298b31030b2731ec5c99ebbd7e05592f45d
tree2e0b12fd5219dcecd1cb562e1b91036fcb513fd8
parent6aca6f0be55be9c2dc7e9457cc9d67bff86e43e0
Fix assertion failure in global-merge with unused ConstantExpr

The global-merge pass was crashing because it assumes that all ConstantExprs
(reached via the global variables that they use) have at least one user.

I haven't worked out a way to test this, as an unused ConstantExpr cannot be
represented by serialised IR, and global-merge can only be run in llc, which
does not run any passes which can make a ConstantExpr dead.

This (reduced to the point of silliness) C code triggers this bug when compiled
for arm-none-eabi at -O1:

  static a = 7;
  static volatile b[10] = {&a};

  c;
  main() {
    c = 0;
    for (; c < 10;)
      printf(b[c]);
  }

Differential Revision: http://reviews.llvm.org/D10314

llvm-svn: 239308
llvm/lib/CodeGen/GlobalMerge.cpp