Review fixes:

Rename isThereAPrimaryInputWithAFilenameAt to isInputPrimary.
Remove spurious delayed_function_body_parsing.
Simply return type of getOutputFilenamesFromCommandLineOrFilelist.
This commit is contained in:
David Ungar
2017-11-28 13:28:10 -08:00
committed by David Ungar
parent 04c16cdebd
commit 6de07d050c
4 changed files with 7 additions and 13 deletions

View File

@@ -283,7 +283,7 @@ private:
/// Returns the output filenames on the command line or in the output
/// filelist. If there
/// were neither -o's nor an output filelist, returns an empty vector.
const std::vector<std::string> &getOutputFilenamesFromCommandLineOrFilelist();
ArrayRef<std::string> getOutputFilenamesFromCommandLineOrFilelist();
bool checkForUnusedOutputPaths() const;
@@ -666,7 +666,7 @@ bool FrontendArgsToOptionsConverter::computeFallbackModuleName() {
// selected".
return false;
}
const std::vector<std::string> &outputFilenames =
ArrayRef<std::string> outputFilenames =
getOutputFilenamesFromCommandLineOrFilelist();
bool isOutputAUniqueOrdinaryFile =
@@ -684,7 +684,7 @@ bool FrontendArgsToOptionsConverter::computeOutputFilenames() {
FrontendOptions::doesActionProduceOutput(Opts.RequestedAction) ||
!FrontendOptions::doesActionProduceTextualOutput(Opts.RequestedAction));
const std::vector<std::string> &outputFilenamesFromCommandLineOrFilelist =
ArrayRef<std::string> outputFilenamesFromCommandLineOrFilelist =
getOutputFilenamesFromCommandLineOrFilelist();
if (outputFilenamesFromCommandLineOrFilelist.size() > 1) {
@@ -882,7 +882,7 @@ void FrontendArgsToOptionsConverter::computeLLVMArgs() {
}
}
const std::vector<std::string> &
ArrayRef<std::string>
FrontendArgsToOptionsConverter::getOutputFilenamesFromCommandLineOrFilelist() {
if (cachedOutputFilenamesFromCommandLineOrFilelist) {
return *cachedOutputFilenamesFromCommandLineOrFilelist;