spatel added a comment. In https://reviews.llvm.org/D36840#851951, @craig.topper wrote: > I think the X86 diffs looks reasonable. Thanks! So both affected targets are approved. Can I take that as approval of the patch? https://reviews.llvm.org/D36840