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

Ahmed Bougacha ahmed.bougacha at gmail.com
Tue Feb 24 11:25:32 PST 2015


I think there are two parts here:

- checking the return value of FoldConstantArithmetic: that LGTM, though tests would of course be appreciated, if possible.
- disabling some optimizations on Opaque constants: that, I'll let Juergen comment on.  I'm opposed to it because of the brittleness, but pragmatically (and there's precedent) that's a reasonable solution.

For your assert failure, I'm fine with just fixing that for now (that would be r230355, correct?).

-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