[PATCH] D72841: [RFC] Add support for pragma float_control, to control precision and exception behavior at the source level

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 11 11:19:11 PDT 2020


rjmccall added a comment.

In D72841#1917340 <https://reviews.llvm.org/D72841#1917340>, @mibintc wrote:

> @rjmccall Since CompoundAssignmentOperator derives from BinaryOperator, it's not simple to add Trailing storage here.  I think I will have to fold CompoundAssignmentOperator into BinaryOperator and then add the 2 extra fields needed by CompoundAssignmentOperator into Trailing storage.  Can you think of a better way?  I worked on Trailing storage for UnaryOperator first and that wasn't too bad, but Binary is a different story.


It's something we deal with occasionally, but it's definitely annoying.  You basically have to test for which concrete class you have and then ask that class for its trailing storage.

Collapsing the types might be okay but could get involved.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72841





More information about the cfe-commits mailing list