We would like to provide the AST node for ConditionalOperator with source location info for the ? and : tokens.
Besides matching our own needs, the addition of these locations will allow for the removal of the following FIXME in lib/Sema/TreeTransform.h:
// FIXM: ? and : locations are broken.
SourceLocation FakeQuestionLoc = E->getCond()->getLocEnd();
SourceLocation FakeColonLoc = E->getFalseExpr()->getLocStart();
Is that OK also from the point of view of clang developers?