Author: Richard Smith
Date: 2020-08-16T15:40:38-07:00
New Revision: ae3067055b33f6ab5657fbae5845cc743b91c299

URL: 
https://github.com/llvm/llvm-project/commit/ae3067055b33f6ab5657fbae5845cc743b91c299
DIFF: 
https://github.com/llvm/llvm-project/commit/ae3067055b33f6ab5657fbae5845cc743b91c299.diff

LOG: Use consistent code for setting FPFeatures from operator constructors.

Added: 
    

Modified: 
    clang/lib/AST/Expr.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp
index f5aecfb7fdae..e3e5e9dbdc62 100644
--- a/clang/lib/AST/Expr.cpp
+++ b/clang/lib/AST/Expr.cpp
@@ -4487,8 +4487,8 @@ BinaryOperator::BinaryOperator(const ASTContext &Ctx, 
Expr *lhs, Expr *rhs,
   SubExprs[LHS] = lhs;
   SubExprs[RHS] = rhs;
   BinaryOperatorBits.HasFPFeatures = FPFeatures.requiresTrailingStorage();
-  if (BinaryOperatorBits.HasFPFeatures)
-    *getTrailingFPFeatures() = FPFeatures;
+  if (hasStoredFPFeatures())
+    setStoredFPFeatures(FPFeatures);
   setDependence(computeDependence(this));
 }
 
@@ -4504,8 +4504,8 @@ BinaryOperator::BinaryOperator(const ASTContext &Ctx, 
Expr *lhs, Expr *rhs,
   SubExprs[LHS] = lhs;
   SubExprs[RHS] = rhs;
   BinaryOperatorBits.HasFPFeatures = FPFeatures.requiresTrailingStorage();
-  if (BinaryOperatorBits.HasFPFeatures)
-    *getTrailingFPFeatures() = FPFeatures;
+  if (hasStoredFPFeatures())
+    setStoredFPFeatures(FPFeatures);
   setDependence(computeDependence(this));
 }
 


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

Reply via email to