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

Ahmed Bougacha ahmed.bougacha at gmail.com
Mon Jan 19 17:05:35 PST 2015


LGTM.
I'm okay with no test if it's tricky to expose, but maybe we can: what happens now on that AArch64 testcase?  Specifically, is there a difference between just this patch, and this patch + the assert fix in http://reviews.llvm.org/D6940?

-Ahmed


http://reviews.llvm.org/D6946

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






More information about the llvm-commits mailing list