diff options
| author | Andrew Trick <atrick@apple.com> | 2011-11-17 23:36:35 +0000 |
|---|---|---|
| committer | Andrew Trick <atrick@apple.com> | 2011-11-17 23:36:35 +0000 |
| commit | 949045864d36579033ddd57a4aac4db3f4ce87bd (patch) | |
| tree | 7aa07ace0b9e4339520a4f6df69f43c03041fe3d /clang/lib/CodeGen/ModuleBuilder.cpp | |
| parent | 651982790a0ec3a9e1f88b2e0d6231e5d62befc7 (diff) | |
| download | llvm-949045864d36579033ddd57a4aac4db3f4ce87bd.zip llvm-949045864d36579033ddd57a4aac4db3f4ce87bd.tar.gz llvm-949045864d36579033ddd57a4aac4db3f4ce87bd.tar.bz2 | |
Fix an overly general check in SimplifyIndvar to handle useless phi cycles.
The right way to check for a binary operation is
cast<BinaryOperator>. The original check: cast<Instruction> &&
numOperands() == 2 would match phi "instructions", leading to an
infinite loop in extreme corner case: a useless phi with operands
[self, constant] that prior optimization passes failed to remove,
being used in the loop by another useless phi, in turn being used by an
lshr or udiv.
Fixes PR11350: runaway iteration assertion.
llvm-svn: 144935
Diffstat (limited to 'clang/lib/CodeGen/ModuleBuilder.cpp')
0 files changed, 0 insertions, 0 deletions