<div dir="ltr">I agree as well. We should make it work well, but we should keep the functionality.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Aug 3, 2014 at 8:30 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br>
<br>
It make sense to ignore temp files, but I also find this useful.<br>
<div class="HOEnZb"><div class="h5"><br>
> On 2014 Aug 1, at 10:34, Owen Anderson <<a href="mailto:resistor@mac.com">resistor@mac.com</a>> wrote:<br>
><br>
> I find this functionality very handy in sanity-checking that I’ve remembered to add new files to the build system.  I’d argue strongly against removing it.  Could it be amended to ignore common backup file patterns?<br>

><br>
> —Owen<br>
><br>
> On Jul 17, 2014, at 11:59 PM, Matt Arsenault <<a href="mailto:Matthew.Arsenault@amd.com">Matthew.Arsenault@amd.com</a>> wrote:<br>
><br>
>> I don't see what value this provides. It does globbing for<br>
>> files to build, and still requires manually listing the files.<br>
>> emacs by default creates backup files in the same directory if you<br>
>> make changes without saving, resulting in really irritating<br>
>> build errors from the "found unknown source file .#Foo.cpp"<br>
>><br>
>> I also don't think it's consistently used, since this<br>
>> doesn't seems to happen for some subset of directories.<br>
>><br>
>> <a href="http://reviews.llvm.org/D4581" target="_blank">http://reviews.llvm.org/D4581</a><br>
>><br>
>> Files:<br>
>> cmake/modules/LLVMProcessSources.cmake<br>
>><br>
>> Index: cmake/modules/LLVMProcessSources.cmake<br>
>> ===================================================================<br>
>> --- cmake/modules/LLVMProcessSources.cmake<br>
>> +++ cmake/modules/LLVMProcessSources.cmake<br>
>> @@ -41,7 +41,6 @@<br>
>> function(llvm_process_sources OUT_VAR)<br>
>>  cmake_parse_arguments(ARG "" "" "ADDITIONAL_HEADERS" ${ARGN})<br>
>>  set(sources ${ARG_UNPARSED_ARGUMENTS})<br>
>> -  llvm_check_source_file_list( ${sources} )<br>
>>  if( MSVC_IDE OR XCODE )<br>
>>    # This adds .td and .h files to the Visual Studio solution:<br>
>>    add_td_sources(sources)<br>
>> @@ -52,20 +51,3 @@<br>
>><br>
>>  set( ${OUT_VAR} ${sources} PARENT_SCOPE )<br>
>> endfunction(llvm_process_sources)<br>
>> -<br>
>> -<br>
>> -function(llvm_check_source_file_list)<br>
>> -  set(listed ${ARGN})<br>
>> -  file(GLOB globbed *.c *.cpp)<br>
>> -  foreach(g ${globbed})<br>
>> -    get_filename_component(fn ${g} NAME)<br>
>> -    list(FIND LLVM_OPTIONAL_SOURCES ${fn} idx)<br>
>> -    if( idx LESS 0 )<br>
>> -      list(FIND listed ${fn} idx)<br>
>> -      if( idx LESS 0 )<br>
>> -        message(SEND_ERROR "Found unknown source file ${g}<br>
>> -Please update ${CMAKE_CURRENT_LIST_FILE}\n")<br>
>> -      endif()<br>
>> -    endif()<br>
>> -  endforeach()<br>
>> -endfunction(llvm_check_source_file_list)<br>
>> <D4581.11635.patch>_______________________________________________<br>
>> llvm-commits mailing list<br>
>> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>