[PATCH] D55395: Re-order content in OMPDeclareReductionDecl dump

[Richard Smith - zygoloid via Phabricator via cfe-commits]

rsmith added a comment.

In D55395#1322244 <https://reviews.llvm.org/D55395#1322244>, @ABataev wrote:

> This is wrong, the original implementation is correct and should not be changed.


The original implementation looks pretty clearly wrong to me, but I think this is wrong too. It looks like what was intended here was probably to label the two children as "combiner" and "initializer" respectively. As-is, the "combiner"  text on the `OMPDeclareReductionDecl` line means nothing at all.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55395/new/

https://reviews.llvm.org/D55395