This is an archive of the discontinued LLVM Phabricator instance.

Differential D135963

Fix build issue due to https://reviews.llvm.org/rG84a9ec2ff1ee97fd7e8ed988f5e7b197aab84a7b
AcceptedPublic

Authored by zahiraam on Oct 14 2022, 8:35 AM.

Download Raw Diff

Details

Reviewers

aaron.ballman

Diff Detail

Event Timeline

zahiraam created this revision.Oct 14 2022, 8:35 AM

Herald added a project: Restricted Project. · View Herald TranscriptOct 14 2022, 8:35 AM

zahiraam requested review of this revision.Oct 14 2022, 8:35 AM

Herald added a project: Restricted Project. · View Herald TranscriptOct 14 2022, 8:35 AM

LGTM -- btw, feel free to land fixes to buildbot breaks without code review.

This revision is now accepted and ready to land.Oct 14 2022, 8:37 AM

In D135963#3858638, @aaron.ballman wrote:

LGTM -- btw, feel free to land fixes to buildbot breaks without code review.

thanks.

Harbormaster completed remote builds in B192183: Diff 467784.Oct 14 2022, 9:04 AM

Revision Contents

Path

Size

clang/

lib/

CodeGen/

CGCall.cpp

4 lines

Diff 467784

clang/lib/CodeGen/CGCall.cpp

Show First 20 Lines • Show All 1,857 Lines • ▼ Show 20 Lines	if (AttrOnCallSite) {
// unsafe/inf/nan/nsz are handled by instruction-level FastMathFlags.		// unsafe/inf/nan/nsz are handled by instruction-level FastMathFlags.
if (LangOpts.NoHonorInfs)		if (LangOpts.NoHonorInfs)
FuncAttrs.addAttribute("no-infs-fp-math", "true");		FuncAttrs.addAttribute("no-infs-fp-math", "true");
if (LangOpts.NoHonorNaNs)		if (LangOpts.NoHonorNaNs)
FuncAttrs.addAttribute("no-nans-fp-math", "true");		FuncAttrs.addAttribute("no-nans-fp-math", "true");
if (LangOpts.ApproxFunc)		if (LangOpts.ApproxFunc)
FuncAttrs.addAttribute("approx-func-fp-math", "true");		FuncAttrs.addAttribute("approx-func-fp-math", "true");
if ((LangOpts.FastMath \|\|		if ((LangOpts.FastMath \|\|
!LangOpts.FastMath && LangOpts.AllowFPReassoc && LangOpts.AllowRecip &&		(!LangOpts.FastMath && LangOpts.AllowFPReassoc && LangOpts.AllowRecip &&
!LangOpts.FiniteMathOnly && LangOpts.NoSignedZero &&		!LangOpts.FiniteMathOnly && LangOpts.NoSignedZero &&
LangOpts.ApproxFunc) &&		LangOpts.ApproxFunc)) &&
LangOpts.getDefaultFPContractMode() != LangOptions::FPModeKind::FPM_Off)		LangOpts.getDefaultFPContractMode() != LangOptions::FPModeKind::FPM_Off)
FuncAttrs.addAttribute("unsafe-fp-math", "true");		FuncAttrs.addAttribute("unsafe-fp-math", "true");
if (CodeGenOpts.SoftFloat)		if (CodeGenOpts.SoftFloat)
FuncAttrs.addAttribute("use-soft-float", "true");		FuncAttrs.addAttribute("use-soft-float", "true");
FuncAttrs.addAttribute("stack-protector-buffer-size",		FuncAttrs.addAttribute("stack-protector-buffer-size",
llvm::utostr(CodeGenOpts.SSPBufferSize));		llvm::utostr(CodeGenOpts.SSPBufferSize));
if (LangOpts.NoSignedZero)		if (LangOpts.NoSignedZero)
FuncAttrs.addAttribute("no-signed-zeros-fp-math", "true");		FuncAttrs.addAttribute("no-signed-zeros-fp-math", "true");
▲ Show 20 Lines • Show All 3,799 Lines • Show Last 20 Lines