Skip to content

Commit f3ecc74

Browse files
naveen-sethnaveen-seth
committed
Remove dependency on clangDriver from flangFrontend
1 parent ac2b233 commit f3ecc74

File tree

8 files changed

+174
-177
lines changed

8 files changed

+174
-177
lines changed

‎clang/include/clang/Driver/CommonArgs.h‎

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -291,16 +291,6 @@ void handleVectorizeLoopsArgs(const llvm::opt::ArgList &Args,
291291
void handleVectorizeSLPArgs(const llvm::opt::ArgList &Args,
292292
llvm::opt::ArgStringList &CmdArgs);
293293

294-
// Parse -mprefer-vector-width=. Return the Value string if well-formed.
295-
// Otherwise, return an empty string and issue a diagnosic message if needed.
296-
StringRef parseMPreferVectorWidthOption(clang::DiagnosticsEngine &Diags,
297-
const llvm::opt::ArgList &Args);
298-
299-
// Parse -mrecip. Return the Value string if well-formed.
300-
// Otherwise, return an empty string and issue a diagnosic message if needed.
301-
StringRef parseMRecipOption(clang::DiagnosticsEngine &Diags,
302-
const llvm::opt::ArgList &Args);
303-
304294
// Convert ComplexRangeKind to a string that can be passed as a frontend option.
305295
std::string complexRangeKindToStr(LangOptions::ComplexRangeKind Range);
306296

‎clang/include/clang/Options/OptionUtils.h‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,16 @@ inline uint64_t getLastArgUInt64Value(const llvm::opt::ArgList &Args,
5353
return getLastArgUInt64Value(Args, Id, Default, &Diags, Base);
5454
}
5555

56+
// Parse -mprefer-vector-width=. Return the Value string if well-formed.
57+
// Otherwise, return an empty string and issue a diagnosic message if needed.
58+
StringRef parseMPreferVectorWidthOption(clang::DiagnosticsEngine &Diags,
59+
const llvm::opt::ArgList &Args);
60+
61+
// Parse -mrecip. Return the Value string if well-formed.
62+
// Otherwise, return an empty string and issue a diagnosic message if needed.
63+
StringRef parseMRecipOption(clang::DiagnosticsEngine &Diags,
64+
const llvm::opt::ArgList &Args);
65+
5666
} // namespace clang
5767

5868
#endif // LLVM_CLANG_OPTIONS_OPTIONUTILS_H

‎clang/lib/Driver/ToolChains/Clang.cpp‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@
3232
#include "clang/Driver/SanitizerArgs.h"
3333
#include "clang/Driver/Types.h"
3434
#include "clang/Driver/XRayArgs.h"
35+
#include "clang/Options/OptionUtils.h"
3536
#include "clang/Options/Options.h"
3637
#include "llvm/ADT/ScopeExit.h"
3738
#include "llvm/ADT/SmallSet.h"

‎clang/lib/Driver/ToolChains/CommonArgs.cpp‎

Lines changed: 0 additions & 163 deletions
Original file line numberDiff line numberDiff line change
@@ -3398,169 +3398,6 @@ void tools::handleInterchangeLoopsArgs(const ArgList &Args,
33983398
CmdArgs.push_back("-floop-interchange");
33993399
}
34003400

3401-
// Parse -mprefer-vector-width=. Return the Value string if well-formed.
3402-
// Otherwise, return an empty string and issue a diagnosic message if needed.
3403-
StringRef tools::parseMPreferVectorWidthOption(clang::DiagnosticsEngine &Diags,
3404-
const llvm::opt::ArgList &Args) {
3405-
Arg *A = Args.getLastArg(options::OPT_mprefer_vector_width_EQ);
3406-
if (!A)
3407-
return "";
3408-
3409-
StringRef Value = A->getValue();
3410-
unsigned Width LLVM_ATTRIBUTE_UNINITIALIZED;
3411-
3412-
// Only "none" and Integer values are accepted by
3413-
// -mprefer-vector-width=<value>.
3414-
if (Value != "none" && Value.getAsInteger(10, Width)) {
3415-
Diags.Report(clang::diag::err_drv_invalid_value)
3416-
<< A->getOption().getName() << Value;
3417-
return "";
3418-
}
3419-
3420-
return Value;
3421-
}
3422-
3423-
// This is a helper function for validating the optional refinement step
3424-
// parameter in reciprocal argument strings. Return false if there is an error
3425-
// parsing the refinement step. Otherwise, return true and set the Position
3426-
// of the refinement step in the input string.
3427-
static bool getRefinementStep(StringRef In, clang::DiagnosticsEngine &Diags,
3428-
const Arg &A, size_t &Position) {
3429-
const char RefinementStepToken = ':';
3430-
Position = In.find(RefinementStepToken);
3431-
if (Position != StringRef::npos) {
3432-
StringRef Option = A.getOption().getName();
3433-
StringRef RefStep = In.substr(Position + 1);
3434-
// Allow exactly one numeric character for the additional refinement
3435-
// step parameter. This is reasonable for all currently-supported
3436-
// operations and architectures because we would expect that a larger value
3437-
// of refinement steps would cause the estimate "optimization" to
3438-
// under-perform the native operation. Also, if the estimate does not
3439-
// converge quickly, it probably will not ever converge, so further
3440-
// refinement steps will not produce a better answer.
3441-
if (RefStep.size() != 1) {
3442-
Diags.Report(diag::err_drv_invalid_value) << Option << RefStep;
3443-
return false;
3444-
}
3445-
char RefStepChar = RefStep[0];
3446-
if (RefStepChar < '0' || RefStepChar > '9') {
3447-
Diags.Report(diag::err_drv_invalid_value) << Option << RefStep;
3448-
return false;
3449-
}
3450-
}
3451-
return true;
3452-
}
3453-
3454-
// Parse -mrecip. Return the Value string if well-formed.
3455-
// Otherwise, return an empty string and issue a diagnosic message if needed.
3456-
StringRef tools::parseMRecipOption(clang::DiagnosticsEngine &Diags,
3457-
const ArgList &Args) {
3458-
StringRef DisabledPrefixIn = "!";
3459-
StringRef DisabledPrefixOut = "!";
3460-
StringRef EnabledPrefixOut = "";
3461-
StringRef Out = "";
3462-
3463-
Arg *A = Args.getLastArg(options::OPT_mrecip, options::OPT_mrecip_EQ);
3464-
if (!A)
3465-
return "";
3466-
3467-
unsigned NumOptions = A->getNumValues();
3468-
if (NumOptions == 0) {
3469-
// No option is the same as "all".
3470-
return "all";
3471-
}
3472-
3473-
// Pass through "all", "none", or "default" with an optional refinement step.
3474-
if (NumOptions == 1) {
3475-
StringRef Val = A->getValue(0);
3476-
size_t RefStepLoc;
3477-
if (!getRefinementStep(Val, Diags, *A, RefStepLoc))
3478-
return "";
3479-
StringRef ValBase = Val.slice(0, RefStepLoc);
3480-
if (ValBase == "all" || ValBase == "none" || ValBase == "default") {
3481-
return Val;
3482-
}
3483-
}
3484-
3485-
// Each reciprocal type may be enabled or disabled individually.
3486-
// Check each input value for validity, concatenate them all back together,
3487-
// and pass through.
3488-
3489-
llvm::StringMap<bool> OptionStrings;
3490-
OptionStrings.insert(std::make_pair("divd", false));
3491-
OptionStrings.insert(std::make_pair("divf", false));
3492-
OptionStrings.insert(std::make_pair("divh", false));
3493-
OptionStrings.insert(std::make_pair("vec-divd", false));
3494-
OptionStrings.insert(std::make_pair("vec-divf", false));
3495-
OptionStrings.insert(std::make_pair("vec-divh", false));
3496-
OptionStrings.insert(std::make_pair("sqrtd", false));
3497-
OptionStrings.insert(std::make_pair("sqrtf", false));
3498-
OptionStrings.insert(std::make_pair("sqrth", false));
3499-
OptionStrings.insert(std::make_pair("vec-sqrtd", false));
3500-
OptionStrings.insert(std::make_pair("vec-sqrtf", false));
3501-
OptionStrings.insert(std::make_pair("vec-sqrth", false));
3502-
3503-
for (unsigned i = 0; i != NumOptions; ++i) {
3504-
StringRef Val = A->getValue(i);
3505-
3506-
bool IsDisabled = Val.starts_with(DisabledPrefixIn);
3507-
// Ignore the disablement token for string matching.
3508-
if (IsDisabled)
3509-
Val = Val.substr(1);
3510-
3511-
size_t RefStep;
3512-
if (!getRefinementStep(Val, Diags, *A, RefStep))
3513-
return "";
3514-
3515-
StringRef ValBase = Val.slice(0, RefStep);
3516-
llvm::StringMap<bool>::iterator OptionIter = OptionStrings.find(ValBase);
3517-
if (OptionIter == OptionStrings.end()) {
3518-
// Try again specifying float suffix.
3519-
OptionIter = OptionStrings.find(ValBase.str() + 'f');
3520-
if (OptionIter == OptionStrings.end()) {
3521-
// The input name did not match any known option string.
3522-
Diags.Report(diag::err_drv_unknown_argument) << Val;
3523-
return "";
3524-
}
3525-
// The option was specified without a half or float or double suffix.
3526-
// Make sure that the double or half entry was not already specified.
3527-
// The float entry will be checked below.
3528-
if (OptionStrings[ValBase.str() + 'd'] ||
3529-
OptionStrings[ValBase.str() + 'h']) {
3530-
Diags.Report(diag::err_drv_invalid_value)
3531-
<< A->getOption().getName() << Val;
3532-
return "";
3533-
}
3534-
}
3535-
3536-
if (OptionIter->second == true) {
3537-
// Duplicate option specified.
3538-
Diags.Report(diag::err_drv_invalid_value)
3539-
<< A->getOption().getName() << Val;
3540-
return "";
3541-
}
3542-
3543-
// Mark the matched option as found. Do not allow duplicate specifiers.
3544-
OptionIter->second = true;
3545-
3546-
// If the precision was not specified, also mark the double and half entry
3547-
// as found.
3548-
if (ValBase.back() != 'f' && ValBase.back() != 'd' &&
3549-
ValBase.back() != 'h') {
3550-
OptionStrings[ValBase.str() + 'd'] = true;
3551-
OptionStrings[ValBase.str() + 'h'] = true;
3552-
}
3553-
3554-
// Build the output string.
3555-
StringRef Prefix = IsDisabled ? DisabledPrefixOut : EnabledPrefixOut;
3556-
Out = Args.MakeArgString(Out + Prefix + Val);
3557-
if (i != NumOptions - 1)
3558-
Out = Args.MakeArgString(Out + ",");
3559-
}
3560-
3561-
return Out;
3562-
}
3563-
35643401
std::string tools::complexRangeKindToStr(LangOptions::ComplexRangeKind Range) {
35653402
switch (Range) {
35663403
case LangOptions::ComplexRangeKind::CX_Full:

‎clang/lib/Driver/ToolChains/Flang.cpp‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111

1212
#include "clang/Basic/CodeGenOptions.h"
1313
#include "clang/Driver/CommonArgs.h"
14+
#include "clang/Options/OptionUtils.h"
1415
#include "clang/Options/Options.h"
1516
#include "llvm/Frontend/Debug/Options.h"
1617
#include "llvm/Support/Path.h"

‎clang/lib/Options/OptionUtils.cpp‎

Lines changed: 160 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
#include "clang/Options/OptionUtils.h"
1010
#include "clang/Basic/Diagnostic.h"
1111
#include "clang/Basic/DiagnosticDriver.h"
12+
#include "clang/Options/Options.h"
1213
#include "llvm/Option/ArgList.h"
1314

1415
using namespace clang;
@@ -44,4 +45,163 @@ uint64_t getLastArgUInt64Value(const ArgList &Args, OptSpecifier Id,
4445
return getLastArgIntValueImpl<uint64_t>(Args, Id, Default, Diags, Base);
4546
}
4647

48+
StringRef parseMPreferVectorWidthOption(clang::DiagnosticsEngine &Diags,
49+
const llvm::opt::ArgList &Args) {
50+
const Arg *A = Args.getLastArg(options::OPT_mprefer_vector_width_EQ);
51+
if (!A)
52+
return "";
53+
54+
StringRef Value = A->getValue();
55+
unsigned Width LLVM_ATTRIBUTE_UNINITIALIZED;
56+
57+
// Only "none" and Integer values are accepted by
58+
// -mprefer-vector-width=<value>.
59+
if (Value != "none" && Value.getAsInteger(10, Width)) {
60+
Diags.Report(clang::diag::err_drv_invalid_value)
61+
<< A->getOption().getName() << Value;
62+
return "";
63+
}
64+
65+
return Value;
66+
}
67+
68+
// This is a helper function for validating the optional refinement step
69+
// parameter in reciprocal argument strings. Return false if there is an error
70+
// parsing the refinement step. Otherwise, return true and set the Position
71+
// of the refinement step in the input string.
72+
static bool getRefinementStep(StringRef In, clang::DiagnosticsEngine &Diags,
73+
const Arg &A, size_t &Position) {
74+
const char RefinementStepToken = ':';
75+
Position = In.find(RefinementStepToken);
76+
if (Position != StringRef::npos) {
77+
StringRef Option = A.getOption().getName();
78+
StringRef RefStep = In.substr(Position + 1);
79+
// Allow exactly one numeric character for the additional refinement
80+
// step parameter. This is reasonable for all currently-supported
81+
// operations and architectures because we would expect that a larger value
82+
// of refinement steps would cause the estimate "optimization" to
83+
// under-perform the native operation. Also, if the estimate does not
84+
// converge quickly, it probably will not ever converge, so further
85+
// refinement steps will not produce a better answer.
86+
if (RefStep.size() != 1) {
87+
Diags.Report(diag::err_drv_invalid_value) << Option << RefStep;
88+
return false;
89+
}
90+
char RefStepChar = RefStep[0];
91+
if (RefStepChar < '0' || RefStepChar > '9') {
92+
Diags.Report(diag::err_drv_invalid_value) << Option << RefStep;
93+
return false;
94+
}
95+
}
96+
return true;
97+
}
98+
99+
StringRef parseMRecipOption(clang::DiagnosticsEngine &Diags,
100+
const ArgList &Args) {
101+
StringRef DisabledPrefixIn = "!";
102+
StringRef DisabledPrefixOut = "!";
103+
StringRef EnabledPrefixOut = "";
104+
StringRef Out = "";
105+
106+
const Arg *A = Args.getLastArg(options::OPT_mrecip, options::OPT_mrecip_EQ);
107+
if (!A)
108+
return "";
109+
110+
const unsigned NumOptions = A->getNumValues();
111+
if (NumOptions == 0) {
112+
// No option is the same as "all".
113+
return "all";
114+
}
115+
116+
// Pass through "all", "none", or "default" with an optional refinement step.
117+
if (NumOptions == 1) {
118+
StringRef Val = A->getValue(0);
119+
size_t RefStepLoc;
120+
if (!getRefinementStep(Val, Diags, *A, RefStepLoc))
121+
return "";
122+
StringRef ValBase = Val.slice(0, RefStepLoc);
123+
if (ValBase == "all" || ValBase == "none" || ValBase == "default") {
124+
return Val;
125+
}
126+
}
127+
128+
// Each reciprocal type may be enabled or disabled individually.
129+
// Check each input value for validity, concatenate them all back together,
130+
// and pass through.
131+
132+
llvm::StringMap<bool> OptionStrings;
133+
OptionStrings.insert(std::make_pair("divd", false));
134+
OptionStrings.insert(std::make_pair("divf", false));
135+
OptionStrings.insert(std::make_pair("divh", false));
136+
OptionStrings.insert(std::make_pair("vec-divd", false));
137+
OptionStrings.insert(std::make_pair("vec-divf", false));
138+
OptionStrings.insert(std::make_pair("vec-divh", false));
139+
OptionStrings.insert(std::make_pair("sqrtd", false));
140+
OptionStrings.insert(std::make_pair("sqrtf", false));
141+
OptionStrings.insert(std::make_pair("sqrth", false));
142+
OptionStrings.insert(std::make_pair("vec-sqrtd", false));
143+
OptionStrings.insert(std::make_pair("vec-sqrtf", false));
144+
OptionStrings.insert(std::make_pair("vec-sqrth", false));
145+
146+
for (unsigned i = 0; i != NumOptions; ++i) {
147+
StringRef Val = A->getValue(i);
148+
149+
bool IsDisabled = Val.starts_with(DisabledPrefixIn);
150+
// Ignore the disablement token for string matching.
151+
if (IsDisabled)
152+
Val = Val.substr(1);
153+
154+
size_t RefStep;
155+
if (!getRefinementStep(Val, Diags, *A, RefStep))
156+
return "";
157+
158+
StringRef ValBase = Val.slice(0, RefStep);
159+
llvm::StringMap<bool>::iterator OptionIter = OptionStrings.find(ValBase);
160+
if (OptionIter == OptionStrings.end()) {
161+
// Try again specifying float suffix.
162+
OptionIter = OptionStrings.find(ValBase.str() + 'f');
163+
if (OptionIter == OptionStrings.end()) {
164+
// The input name did not match any known option string.
165+
Diags.Report(diag::err_drv_unknown_argument) << Val;
166+
return "";
167+
}
168+
// The option was specified without a half or float or double suffix.
169+
// Make sure that the double or half entry was not already specified.
170+
// The float entry will be checked below.
171+
if (OptionStrings[ValBase.str() + 'd'] ||
172+
OptionStrings[ValBase.str() + 'h']) {
173+
Diags.Report(diag::err_drv_invalid_value)
174+
<< A->getOption().getName() << Val;
175+
return "";
176+
}
177+
}
178+
179+
if (OptionIter->second == true) {
180+
// Duplicate option specified.
181+
Diags.Report(diag::err_drv_invalid_value)
182+
<< A->getOption().getName() << Val;
183+
return "";
184+
}
185+
186+
// Mark the matched option as found. Do not allow duplicate specifiers.
187+
OptionIter->second = true;
188+
189+
// If the precision was not specified, also mark the double and half entry
190+
// as found.
191+
if (ValBase.back() != 'f' && ValBase.back() != 'd' &&
192+
ValBase.back() != 'h') {
193+
OptionStrings[ValBase.str() + 'd'] = true;
194+
OptionStrings[ValBase.str() + 'h'] = true;
195+
}
196+
197+
// Build the output string.
198+
StringRef Prefix = IsDisabled ? DisabledPrefixOut : EnabledPrefixOut;
199+
Out = Args.MakeArgString(Out + Prefix + Val);
200+
if (i != NumOptions - 1)
201+
Out = Args.MakeArgString(Out + ",");
202+
}
203+
204+
return Out;
205+
}
206+
47207
} // namespace clang

‎flang/lib/Frontend/CMakeLists.txt‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,6 @@ add_flang_library(flangFrontend
7575

7676
CLANG_LIBS
7777
clangBasic
78-
clangDriver
7978
clangOptions
8079
)
8180

0 commit comments

Comments
 (0)