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

Ahmed Bougacha ahmed.bougacha at gmail.com
Tue Feb 24 10:33:02 PST 2015


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/






More information about the llvm-commits mailing list