[PATCH] DAGCombiner: Continue combining if FoldConstantArithmetic() fails.

[Ahmed Bougacha]

Fair enough, reordering the optimizations isn't a solution.  Looking through old commits, I see there are a few that do exactly this: add isOpaque checks around DAG combines.
Having a separate node type for opaque constants does sound less brittle;  any idea of what would be involved?

-Ahmed


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D6946

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/