[PATCH] D76384: Move FPFeatures from BinaryOperator bitfields to Trailing storage

Melanie Blower via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 23 07:03:35 PDT 2020


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





More information about the cfe-commits mailing list