hubert-reinterpretcast wrote: > To clarify the purpose of floating-point status checks in AST and ByteCode > evaluators during translation (as opposed to mandatory constant expression > evaluation): > > * Rename checkFloatingPointResult to > checkFloatingPointResultForOpportunisticFolding in ExprConstant.cpp > > * Update its Doxygen documentation comment > > * Refactor CheckFloatResult in Interp.cpp to delegate to CheckFloatStatus.
@lntue, please update the PR title and description for the new changes. Also, the description should describe `CheckFloatStatus` as being split out from `CheckFloatResult` (and explain why that makes sense). https://github.com/llvm/llvm-project/pull/213750 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
