[PATCH] D74436: Change clang option -ffp-model=precise to select ffp-contract=on

Blower, Melanie I via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 14 07:34:17 PST 2020


I reverted MaskRay's "reland" since the original patch is causing trouble on PowerPC, check-all is passing on my box.  Sorry for the trouble. 

> -----Original Message-----
> From: Andy Kaylor via Phabricator <reviews at reviews.llvm.org>
> Sent: Thursday, February 13, 2020 9:20 PM
> To: Blower, Melanie I <melanie.blower at intel.com>; lebedev.ri at gmail.com;
> rjmccall at gmail.com; sepavloff at gmail.com
> Cc: maskray at google.com; jji at us.ibm.com; david.bolvansky at gmail.com;
> martin at martin.st; Wang, Pengfei <pengfei.wang at intel.com>;
> wuzish at cn.ibm.com; nemanja.i.ibm at gmail.com; kit.barton at gmail.com; cfe-
> commits at lists.llvm.org; mlekena at skidmore.edu; blitzrakete at gmail.com;
> shenhan at google.com; peter.waller at arm.com
> Subject: [PATCH] D74436: Change clang option -ffp-model=precise to select ffp-
> contract=on
> 
> andrew.w.kaylor added a subscriber: MaskRay.
> andrew.w.kaylor added a comment.
> 
> In D74436#1875386 <https://reviews.llvm.org/D74436#1875386>, @thakis
> wrote:
> 
> > The revert of this breaks tests everywhere, as far as I can tell.
> 
> 
> It looks like something strange happened with the revert:
> 
> > clang-11: warning: overriding '-ffp-model=strict' option with '-ffp-
> model=strict' [-Woverriding-t-option]
> 
> I believe the problem is that the original change that was being reverted
> contained this:
> 
>   clang/lib/Driver/ToolChains/Clang.cpp
>   @@ -2768,7 +2766,7 @@ static void RenderFloatingPointOptions(const
> ToolChain &TC, const Driver &D,
>           !AssociativeMath && !ReciprocalMath &&
>           SignedZeros && TrappingMath && RoundingFPMath &&
>           DenormalFPMath != llvm::DenormalMode::getIEEE() &&
>   +        FPContract.empty())
>   -        (FPContract.equals("off") || FPContract.empty()))
> 
> But sometime in the land-revert-land-revert cycle the line above that changed,
> causing the merge to miss this change in the most recent revert. I see that
> @MaskRay has since re-landed this change set, but it's going to cause problems
> for PowerPC. If someone needs to revert this yet again, I think it can be safely
> done by recovering the change above.
> 
> Apologies for the mess!
> 
> 
> Repository:
>   rG LLVM Github Monorepo
> 
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D74436/new/
> 
> https://reviews.llvm.org/D74436
> 
> 



More information about the cfe-commits mailing list