mibintc added a comment.

In D76384#1934785 <https://reviews.llvm.org/D76384#1934785>, @rjmccall wrote:

> Let's make this patch be purely a representation change.  I wouldn't expect 
> *any* test changes from that; if there are, we should understand why.  You 
> can then add the stuff about tracking whether there's a pragma in a separate 
> patch.


OK, some of the tests show ConditionalOperator in the IR matching. Since that 
operator is necessarily getting merged into BinaryOperator those tests will 
have to change. I'll see about changing the srcpos for the conditional 
assignment operator so i can undo the srcpos changes in those tests.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76384



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to