[PATCH] DAGCombiner: Continue combining if FoldConstantArithmetic() fails.
Matthias Braun
matze at braunis.de
Tue Feb 24 10:52:03 PST 2015
That would obviously involve changing all targets to recognize that new node type, in all sorts of different addressing modes. I also don't know whether all of this was already discussed when OpaqueConstants were introduced. Anyway I just wanted to fix a few bugs here and not rewrite all this stuff, so this patch won't get a LGTM unless it's a rewrite of the whole OpaqueConstants concepts?
- Matthias
> On Feb 24, 2015, at 10:33 AM, Ahmed Bougacha <ahmed.bougacha at gmail.com> wrote:
>
> 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/
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list