<div dir="ltr">It might have been that the problem, yes. Looking at the included commits, there's a gap between the last llvm one this build had, and the last commit on clang.<div><br></div><div>Right now the bot has updated some more times and is green. Sorry about the noise.</div><div><br></div><div class="gmail_extra">Thanks!</div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"> Filipe<br></div></div>
<br><div class="gmail_quote">On Tue, Mar 17, 2015 at 2:10 PM, Michael Zolotukhin <span dir="ltr"><<a href="mailto:mzolotukhin@apple.com" target="_blank">mzolotukhin@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi Filipe,<div><br></div><div>Could it be that clang repo is newer, than the llvm repo?</div><div><br></div><div>From the error messages it looks like clang-repo has been updated, while the llvm-repo is missing one commit.</div><div><br></div><div>Thanks,</div><div>Michael</div><div><div class="h5"><div><br><div><blockquote type="cite"><div>On Mar 17, 2015, at 1:43 PM, Filipe Cabecinhas <<a href="mailto:filcab@gmail.com" target="_blank">filcab@gmail.com</a>> wrote:</div><br><div><div dir="ltr">Hi Michael,<div><br></div><div>This breaks Windows, I guess you need to explicitly name the enum:</div><div><div>C:\Buildbot\Slave\builds\1.LLVM.Windows.Phase.4\clang.src\lib\CodeGen\BackendUtil.cpp(243): error C2039: 'addVectorizableFunctionsFromVecLib' : is not a member of 'llvm::TargetLibraryInfoImpl' [C:\Buildbot\Slave\builds\1.LLVM.Windows.Phase.4\llvm.obj\tools\clang\lib\CodeGen\clangCodeGen.vcxproj]</div><div> C:\Buildbot\Slave\builds\1.LLVM.Windows.Phase.4\llvm.src\include\llvm/Analysis/TargetLibraryInfo.h(38) : see declaration of 'llvm::TargetLibraryInfoImpl'</div><div>C:\Buildbot\Slave\builds\1.LLVM.Windows.Phase.4\clang.src\lib\CodeGen\BackendUtil.cpp(243): error C2039: 'Accelerate' : is not a member of 'llvm::TargetLibraryInfoImpl' [C:\Buildbot\Slave\builds\1.LLVM.Windows.Phase.4\llvm.obj\tools\clang\lib\CodeGen\clangCodeGen.vcxproj]</div><div> C:\Buildbot\Slave\builds\1.LLVM.Windows.Phase.4\llvm.src\include\llvm/Analysis/TargetLibraryInfo.h(38) : see declaration of 'llvm::TargetLibraryInfoImpl'</div><div>C:\Buildbot\Slave\builds\1.LLVM.Windows.Phase.4\clang.src\lib\CodeGen\BackendUtil.cpp(243): error C2065: 'Accelerate' : undeclared identifier [C:\Buildbot\Slave\builds\1.LLVM.Windows.Phase.4\llvm.obj\tools\clang\lib\CodeGen\clangCodeGen.vcxproj]</div></div><div><br></div><div class="gmail_extra">Thanks,</div><div class="gmail_extra"><br clear="all"><div><div> Filipe<br></div></div>
<br><div class="gmail_quote">On Tue, Mar 17, 2015 at 1:03 PM, Michael Zolotukhin <span dir="ltr"><<a href="mailto:mzolotukhin@apple.com" target="_blank">mzolotukhin@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: mzolotukhin<br>
Date: Tue Mar 17 15:03:11 2015<br>
New Revision: 232533<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=232533&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=232533&view=rev</a><br>
Log:<br>
Add fveclib option.<br>
<br>
Review: <a href="http://reviews.llvm.org/D8097" target="_blank">http://reviews.llvm.org/D8097</a><br>
<br>
Added:<br>
cfe/trunk/test/Driver/fveclib.c<br>
Modified:<br>
cfe/trunk/include/clang/Driver/Options.td<br>
cfe/trunk/include/clang/Frontend/CodeGenOptions.def<br>
cfe/trunk/include/clang/Frontend/CodeGenOptions.h<br>
cfe/trunk/lib/CodeGen/BackendUtil.cpp<br>
cfe/trunk/lib/Driver/Tools.cpp<br>
cfe/trunk/lib/Frontend/CompilerInvocation.cpp<br>
<br>
Modified: cfe/trunk/include/clang/Driver/Options.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=232533&r1=232532&r2=232533&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Options.td?rev=232533&r1=232532&r2=232533&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/include/clang/Driver/Options.td (original)<br>
+++ cfe/trunk/include/clang/Driver/Options.td Tue Mar 17 15:03:11 2015<br>
@@ -756,6 +756,8 @@ def fno_exceptions : Flag<["-"], "fno-ex<br>
def fno_gnu_keywords : Flag<["-"], "fno-gnu-keywords">, Group<f_Group>, Flags<[CC1Option]>;<br>
def fno_inline_functions : Flag<["-"], "fno-inline-functions">, Group<f_clang_Group>, Flags<[CC1Option]>;<br>
def fno_inline : Flag<["-"], "fno-inline">, Group<f_clang_Group>, Flags<[CC1Option]>;<br>
+def fveclib : Joined<["-"], "fveclib=">, Group<f_Group>, Flags<[CC1Option]>,<br>
+ HelpText<"Use the given vector functions library">;<br>
def fno_lax_vector_conversions : Flag<["-"], "fno-lax-vector-conversions">, Group<f_Group>,<br>
HelpText<"Disallow implicit conversions between vectors with a different number of elements or different element types">, Flags<[CC1Option]>;<br>
def fno_merge_all_constants : Flag<["-"], "fno-merge-all-constants">, Group<f_Group>,<br>
<br>
Modified: cfe/trunk/include/clang/Frontend/CodeGenOptions.def<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Frontend/CodeGenOptions.def?rev=232533&r1=232532&r2=232533&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Frontend/CodeGenOptions.def?rev=232533&r1=232532&r2=232533&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/include/clang/Frontend/CodeGenOptions.def (original)<br>
+++ cfe/trunk/include/clang/Frontend/CodeGenOptions.def Tue Mar 17 15:03:11 2015<br>
@@ -162,6 +162,9 @@ VALUE_CODEGENOPT(DwarfVersion, 3, 0)<br>
/// The kind of inlining to perform.<br>
ENUM_CODEGENOPT(Inlining, InliningMethod, 2, NoInlining)<br>
<br>
+// Vector functions library to use.<br>
+ENUM_CODEGENOPT(VecLib, VectorLibrary, 1, NoLibrary)<br>
+<br>
/// The default TLS model to use.<br>
ENUM_CODEGENOPT(DefaultTLSModel, TLSModel, 2, GeneralDynamicTLSModel)<br>
<br>
<br>
Modified: cfe/trunk/include/clang/Frontend/CodeGenOptions.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Frontend/CodeGenOptions.h?rev=232533&r1=232532&r2=232533&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Frontend/CodeGenOptions.h?rev=232533&r1=232532&r2=232533&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/include/clang/Frontend/CodeGenOptions.h (original)<br>
+++ cfe/trunk/include/clang/Frontend/CodeGenOptions.h Tue Mar 17 15:03:11 2015<br>
@@ -46,6 +46,11 @@ public:<br>
OnlyAlwaysInlining // Only run the always inlining pass.<br>
};<br>
<br>
+ enum VectorLibrary {<br>
+ NoLibrary, // Don't use any vector library.<br>
+ Accelerate // Use the Accelerate framework.<br>
+ };<br>
+<br>
enum ObjCDispatchMethodKind {<br>
Legacy = 0,<br>
NonLegacy = 1,<br>
<br>
Modified: cfe/trunk/lib/CodeGen/BackendUtil.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/BackendUtil.cpp?rev=232533&r1=232532&r2=232533&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/BackendUtil.cpp?rev=232533&r1=232532&r2=232533&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/lib/CodeGen/BackendUtil.cpp (original)<br>
+++ cfe/trunk/lib/CodeGen/BackendUtil.cpp Tue Mar 17 15:03:11 2015<br>
@@ -237,6 +237,14 @@ static TargetLibraryInfoImpl *createTLII<br>
TargetLibraryInfoImpl *TLII = new TargetLibraryInfoImpl(TargetTriple);<br>
if (!CodeGenOpts.SimplifyLibCalls)<br>
TLII->disableAllFunctions();<br>
+<br>
+ switch (CodeGenOpts.getVecLib()) {<br>
+ case CodeGenOptions::Accelerate:<br>
+ TLII->addVectorizableFunctionsFromVecLib(TargetLibraryInfoImpl::Accelerate);<br>
+ break;<br>
+ default:<br>
+ break;<br>
+ }<br>
return TLII;<br>
}<br>
<br>
<br>
Modified: cfe/trunk/lib/Driver/Tools.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=232533&r1=232532&r2=232533&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=232533&r1=232532&r2=232533&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/lib/Driver/Tools.cpp (original)<br>
+++ cfe/trunk/lib/Driver/Tools.cpp Tue Mar 17 15:03:11 2015<br>
@@ -2864,6 +2864,8 @@ void Clang::ConstructJob(Compilation &C,<br>
else<br>
CmdArgs.push_back(Args.MakeArgString(getToolChain().getThreadModel()));<br>
<br>
+ Args.AddLastArg(CmdArgs, options::OPT_fveclib);<br>
+<br>
if (!Args.hasFlag(options::OPT_fmerge_all_constants,<br>
options::OPT_fno_merge_all_constants))<br>
CmdArgs.push_back("-fno-merge-all-constants");<br>
<br>
Modified: cfe/trunk/lib/Frontend/CompilerInvocation.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=232533&r1=232532&r2=232533&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=232533&r1=232532&r2=232533&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/lib/Frontend/CompilerInvocation.cpp (original)<br>
+++ cfe/trunk/lib/Frontend/CompilerInvocation.cpp Tue Mar 17 15:03:11 2015<br>
@@ -367,6 +367,16 @@ static bool ParseCodeGenArgs(CodeGenOpti<br>
Opts.setInlining(Args.hasArg(OPT_fno_inline_functions) ?<br>
CodeGenOptions::OnlyAlwaysInlining : Opts.getInlining());<br>
<br>
+ if (Arg *A = Args.getLastArg(OPT_fveclib)) {<br>
+ StringRef Name = A->getValue();<br>
+ if (Name == "Accelerate")<br>
+ Opts.setVecLib(CodeGenOptions::Accelerate);<br>
+ else if (Name == "none")<br>
+ Opts.setVecLib(CodeGenOptions::NoLibrary);<br>
+ else<br>
+ Diags.Report(diag::err_drv_invalid_value) << A->getAsString(Args) << Name;<br>
+ }<br>
+<br>
if (Args.hasArg(OPT_gline_tables_only)) {<br>
Opts.setDebugInfo(CodeGenOptions::DebugLineTablesOnly);<br>
} else if (Args.hasArg(OPT_g_Flag) || Args.hasArg(OPT_gdwarf_2) ||<br>
<br>
Added: cfe/trunk/test/Driver/fveclib.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/fveclib.c?rev=232533&view=auto" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/fveclib.c?rev=232533&view=auto</a><br>
==============================================================================<br>
--- cfe/trunk/test/Driver/fveclib.c (added)<br>
+++ cfe/trunk/test/Driver/fveclib.c Tue Mar 17 15:03:11 2015<br>
@@ -0,0 +1,8 @@<br>
+// RUN: %clang -### -c -fveclib=none %s 2>&1 | FileCheck -check-prefix CHECK-NOLIB %s<br>
+// RUN: %clang -### -c -fveclib=Accelerate %s 2>&1 | FileCheck -check-prefix CHECK-ACCELERATE %s<br>
+// RUN: not %clang -c -fveclib=something %s 2>&1 | FileCheck -check-prefix CHECK-INVALID %s<br>
+<br>
+// CHECK-NOLIB: "-fveclib=none"<br>
+// CHECK-ACCELERATE: "-fveclib=Accelerate"<br>
+<br>
+// CHECK-INVALID: error: invalid value 'something' in '-fveclib=something'<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>
</div></blockquote></div><br></div></div></div></div></blockquote></div><br></div></div>