Re: r355858 - [OPENMP]Fix codegen for declare target link in target regions.

2019-03-11 Thread Azhar Mohammed via cfe-commits
Hi Alexey

Looks like this is failing on Darwin, can you please take a look? Refer to 
http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-incremental/58951/consoleFull
 
.

 TEST 'Clang :: Modules/builtins.m' FAILED 

Script:
--
: 'RUN: at line 1';   rm -rf 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Modules/Output/builtins.m.tmp
: 'RUN: at line 2';   
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/bin/clang
 -cc1 -internal-isystem 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/lib/clang/9.0.0/include
 -nostdsysteminc 
-fmodules-cache-path=/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Modules/Output/builtins.m.tmp
 -fmodules -fimplicit-module-maps -I 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/Inputs
 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/builtins.m
 -verify
: 'RUN: at line 3';   
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/bin/clang
 -cc1 -internal-isystem 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/lib/clang/9.0.0/include
 -nostdsysteminc 
-fmodules-cache-path=/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Modules/Output/builtins.m.tmp
 -fmodules -fimplicit-module-maps -I 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/Inputs
 -x c 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/builtins.m
 -verify
: 'RUN: at line 4';   
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/bin/clang
 -cc1 -internal-isystem 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/lib/clang/9.0.0/include
 -nostdsysteminc 
-fmodules-cache-path=/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Modules/Output/builtins.m.tmp
 -fmodules -fimplicit-module-maps -I 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/Inputs
 -x objective-c++ 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/builtins.m
 -verify
: 'RUN: at line 6';   rm -rf 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Modules/Output/builtins.m.tmp.pch.cache
: 'RUN: at line 7';   
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/bin/clang
 -cc1 -internal-isystem 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/lib/clang/9.0.0/include
 -nostdsysteminc 
-fmodules-cache-path=/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Modules/Output/builtins.m.tmp.pch.cache
 -fmodules -fimplicit-module-maps -I 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/Inputs
 -emit-pch -o 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Modules/Output/builtins.m.tmp.pch
 -x objective-c-header 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/Inputs/use-builtin.h
: 'RUN: at line 8';   
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/bin/clang
 -cc1 -internal-isystem 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/lib/clang/9.0.0/include
 -nostdsysteminc 
-fmodules-cache-path=/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Modules/Output/builtins.m.tmp.pch.cache
 -fmodules -fimplicit-module-maps -I 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/Inputs
 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/builtins.m
 -include-pch 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Modules/Output/builtins.m.tmp.pch
 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/test/Modules/builtins.m
 -verify
--
Exit Code: 134

Command Output (stderr):
--
 <>Assertion failed: ((Imported == nullptr || Imported == 
SuggestedModule.getModule()) && "the imported module is different than the 
suggested one"), function HandleIncludeDirective, file 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm/tools/clang/lib/Lex/PPDirectives.cpp,
 line 1952.

> On Mar 11, 2019, at 12:51 PM, Alexey Bataev via cfe-commits 

Re: [clang-tools-extra] r366458 - [clangd] Refactor background-index shard loading

2019-07-18 Thread Azhar Mohammed via cfe-commits
Hi Kadir

This change is causing test failures, can you please look into it. Refer to 
http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/58104/testReport/.
 

Assertion failed: (TUsIt != FileToTU.end() && "No TU registered for the 
shard"), function takeResult, file 
/Users/buildslave/jenkins/workspace/clang-stage1-configure-RA/llvm/tools/clang/tools/extra/clangd/index/BackgroundIndexLoader.cpp,
 line 131.


Failing Tests (10):
Clangd :: did-change-configuration-params.test
Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.CmdLineHash
Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.DirectIncludesTest
Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.IndexTwoFiles
Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.NoCrashOnErrorFile
Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.NoDotsInAbsPath
Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.ShardStorageEmptyFile
Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.ShardStorageLoad
Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.ShardStorageTest
Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.UncompilableFiles


> On Jul 18, 2019, at 9:25 AM, Kadir Cetinkaya via cfe-commits 
>  wrote:
> 
> Author: kadircet
> Date: Thu Jul 18 09:25:36 2019
> New Revision: 366458
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=366458&view=rev
> Log:
> [clangd] Refactor background-index shard loading
> 
> Reviewers: sammccall
> 
> Subscribers: mgorny, ilya-biryukov, MaskRay, jkorous, arphaman, cfe-commits
> 
> Tags: #clang
> 
> Differential Revision: https://reviews.llvm.org/D64712
> 
> Added:
>clang-tools-extra/trunk/clangd/index/BackgroundIndexLoader.cpp
>clang-tools-extra/trunk/clangd/index/BackgroundIndexLoader.h
> Modified:
>clang-tools-extra/trunk/clangd/CMakeLists.txt
>clang-tools-extra/trunk/clangd/index/Background.cpp
>clang-tools-extra/trunk/clangd/index/Background.h
>clang-tools-extra/trunk/clangd/index/BackgroundRebuild.cpp
>clang-tools-extra/trunk/clangd/index/BackgroundRebuild.h
>clang-tools-extra/trunk/clangd/unittests/BackgroundIndexTests.cpp
> 
> Modified: clang-tools-extra/trunk/clangd/CMakeLists.txt
> URL: 
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/CMakeLists.txt?rev=366458&r1=366457&r2=366458&view=diff
> ==
> --- clang-tools-extra/trunk/clangd/CMakeLists.txt (original)
> +++ clang-tools-extra/trunk/clangd/CMakeLists.txt Thu Jul 18 09:25:36 2019
> @@ -73,6 +73,7 @@ add_clang_library(clangDaemon
>   XRefs.cpp
> 
>   index/Background.cpp
> +  index/BackgroundIndexLoader.cpp
>   index/BackgroundIndexStorage.cpp
>   index/BackgroundQueue.cpp
>   index/BackgroundRebuild.cpp
> 
> Modified: clang-tools-extra/trunk/clangd/index/Background.cpp
> URL: 
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/index/Background.cpp?rev=366458&r1=366457&r2=366458&view=diff
> ==
> --- clang-tools-extra/trunk/clangd/index/Background.cpp (original)
> +++ clang-tools-extra/trunk/clangd/index/Background.cpp Thu Jul 18 09:25:36 
> 2019
> @@ -10,6 +10,7 @@
> #include "ClangdUnit.h"
> #include "Compiler.h"
> #include "Context.h"
> +#include "FSProvider.h"
> #include "Headers.h"
> #include "Logger.h"
> #include "Path.h"
> @@ -18,6 +19,7 @@
> #include "Threading.h"
> #include "Trace.h"
> #include "URI.h"
> +#include "index/BackgroundIndexLoader.h"
> #include "index/FileIndex.h"
> #include "index/IndexAction.h"
> #include "index/MemIndex.h"
> @@ -28,6 +30,8 @@
> #include "clang/Basic/SourceLocation.h"
> #include "clang/Basic/SourceManager.h"
> #include "clang/Driver/Types.h"
> +#include "llvm/ADT/ArrayRef.h"
> +#include "llvm/ADT/DenseSet.h"
> #include "llvm/ADT/Hashing.h"
> #include "llvm/ADT/STLExtras.h"
> #include "llvm/ADT/ScopeExit.h"
> @@ -42,6 +46,7 @@
> #include 
> #include 
> #include 
> +#include 
> #include 
> #include 
> #include 
> @@ -49,6 +54,8 @@
> #include 
> #include 
> #include 
> +#include 
> +#include 
> 
> namespace clang {
> namespace clangd {
> @@ -119,6 +126,18 @@ llvm::SmallString<128> getAbsolutePath(c
>   }
>   return AbsolutePath;
> }
> +
> +bool shardIsStale(const LoadedShard &LS, llvm::vfs::FileSystem *FS) {
> +  auto Buf = FS->getBufferForFile(LS.AbsolutePath);
> +  if (!Buf) {
> +elog("Background-index: Couldn't read {0} to validate stored index: {1}",
> + LS.AbsolutePath, Buf.getError().message());
> +// There is no point in indexing an unreadable file.
> +return false;
> +  }
> +  return digest(Buf->get()->getBuffer()) != LS.Digest;
> +}
> +
> } // namespace
> 
> BackgroundIndex::BackgroundIndex(
> @@ -156,7 +175,7 @@ BackgroundQueue::Task BackgroundIndex::c
> log("Enqueueing {0} commands for indexing", ChangedFiles.size());
> SPAN_ATTACH(Tracer, "files", int64_t(ChangedFiles.size()));
> 
> -au

Re: [clang-tools-extra] r366458 - [clangd] Refactor background-index shard loading

2019-07-19 Thread Azhar Mohammed via cfe-commits
Reverted in r366551. 


Revert r366458, r366467 and r366468

r366458 is causing test failures. r366467 and r366468 had to be reverted as
they were casuing conflict while reverting r366458.

r366468 [clangd] Remove dead code from BackgroundIndex
r366467 [clangd] BackgroundIndex stores shards to the closest project
r366458 [clangd] Refactor background-index shard loading

> On Jul 18, 2019, at 6:21 PM, Azhar Mohammed  wrote:
> 
> Hi Kadir
> 
> This change is causing test failures, can you please look into it. Refer to 
> http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/58104/testReport/
>  
> .
>  
> 
> Assertion failed: (TUsIt != FileToTU.end() && "No TU registered for the 
> shard"), function takeResult, file 
> /Users/buildslave/jenkins/workspace/clang-stage1-configure-RA/llvm/tools/clang/tools/extra/clangd/index/BackgroundIndexLoader.cpp,
>  line 131.
> 
> 
> Failing Tests (10):
> Clangd :: did-change-configuration-params.test
> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.CmdLineHash
> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.DirectIncludesTest
> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.IndexTwoFiles
> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.NoCrashOnErrorFile
> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.NoDotsInAbsPath
> Clangd Unit Tests :: 
> ./ClangdTests/BackgroundIndexTest.ShardStorageEmptyFile
> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.ShardStorageLoad
> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.ShardStorageTest
> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.UncompilableFiles
> 
> 
>> On Jul 18, 2019, at 9:25 AM, Kadir Cetinkaya via cfe-commits 
>> mailto:cfe-commits@lists.llvm.org>> wrote:
>> 
>> Author: kadircet
>> Date: Thu Jul 18 09:25:36 2019
>> New Revision: 366458
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=366458&view=rev 
>> 
>> Log:
>> [clangd] Refactor background-index shard loading
>> 
>> Reviewers: sammccall
>> 
>> Subscribers: mgorny, ilya-biryukov, MaskRay, jkorous, arphaman, cfe-commits
>> 
>> Tags: #clang
>> 
>> Differential Revision: https://reviews.llvm.org/D64712 
>> 
>> 
>> Added:
>>clang-tools-extra/trunk/clangd/index/BackgroundIndexLoader.cpp
>>clang-tools-extra/trunk/clangd/index/BackgroundIndexLoader.h
>> Modified:
>>clang-tools-extra/trunk/clangd/CMakeLists.txt
>>clang-tools-extra/trunk/clangd/index/Background.cpp
>>clang-tools-extra/trunk/clangd/index/Background.h
>>clang-tools-extra/trunk/clangd/index/BackgroundRebuild.cpp
>>clang-tools-extra/trunk/clangd/index/BackgroundRebuild.h
>>clang-tools-extra/trunk/clangd/unittests/BackgroundIndexTests.cpp
>> 
>> Modified: clang-tools-extra/trunk/clangd/CMakeLists.txt
>> URL: 
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/CMakeLists.txt?rev=366458&r1=366457&r2=366458&view=diff
>>  
>> 
>> ==
>> --- clang-tools-extra/trunk/clangd/CMakeLists.txt (original)
>> +++ clang-tools-extra/trunk/clangd/CMakeLists.txt Thu Jul 18 09:25:36 2019
>> @@ -73,6 +73,7 @@ add_clang_library(clangDaemon
>>   XRefs.cpp
>> 
>>   index/Background.cpp
>> +  index/BackgroundIndexLoader.cpp
>>   index/BackgroundIndexStorage.cpp
>>   index/BackgroundQueue.cpp
>>   index/BackgroundRebuild.cpp
>> 
>> Modified: clang-tools-extra/trunk/clangd/index/Background.cpp
>> URL: 
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/index/Background.cpp?rev=366458&r1=366457&r2=366458&view=diff
>>  
>> 
>> ==
>> --- clang-tools-extra/trunk/clangd/index/Background.cpp (original)
>> +++ clang-tools-extra/trunk/clangd/index/Background.cpp Thu Jul 18 09:25:36 
>> 2019
>> @@ -10,6 +10,7 @@
>> #include "ClangdUnit.h"
>> #include "Compiler.h"
>> #include "Context.h"
>> +#include "FSProvider.h"
>> #include "Headers.h"
>> #include "Logger.h"
>> #include "Path.h"
>> @@ -18,6 +19,7 @@
>> #include "Threading.h"
>> #include "Trace.h"
>> #include "URI.h"
>> +#include "index/BackgroundIndexLoader.h"
>> #include "index/FileIndex.h"
>> #include "index/IndexAction.h"
>> #include "index/MemIndex.h"
>> @@ -28,6 +30,8 @@
>> #include "clang/Basic/SourceLocation.h"
>> #include "clang/Basic/SourceManager.h"
>> #include "clang/Driver/Types.h"
>> +#include "llvm/ADT/ArrayRef.h"
>> +#include "llvm/ADT/DenseSet.h"
>> #include "llvm/ADT/Hashing.h"
>> #include "llvm/ADT/STLExtras

Re: [clang-tools-extra] r366458 - [clangd] Refactor background-index shard loading

2019-07-19 Thread Azhar Mohammed via cfe-commits
Looks good now. Thanks Kadir!

> On Jul 19, 2019, at 3:18 AM, Kadir Çetinkaya  wrote:
> 
> Hi Azhar, D64980 should fix the problem. I am reverting your revert while 
> adding the fix in r366559.
> 
> On Fri, Jul 19, 2019 at 11:29 AM Azhar Mohammed  > wrote:
> Reverted in r366551. 
> 
> 
> Revert r366458, r366467 and r366468
> 
> r366458 is causing test failures. r366467 and r366468 had to be reverted 
> as
> they were casuing conflict while reverting r366458.
> 
> r366468 [clangd] Remove dead code from BackgroundIndex
> r366467 [clangd] BackgroundIndex stores shards to the closest project
> r366458 [clangd] Refactor background-index shard loading
> 
>> On Jul 18, 2019, at 6:21 PM, Azhar Mohammed > > wrote:
>> 
>> Hi Kadir
>> 
>> This change is causing test failures, can you please look into it. Refer to 
>> http://green.lab.llvm.org/green/job/clang-stage1-configure-RA/58104/testReport/
>>  
>> .
>>  
>> 
>> Assertion failed: (TUsIt != FileToTU.end() && "No TU registered for the 
>> shard"), function takeResult, file 
>> /Users/buildslave/jenkins/workspace/clang-stage1-configure-RA/llvm/tools/clang/tools/extra/clangd/index/BackgroundIndexLoader.cpp,
>>  line 131.
>> 
>> 
>> Failing Tests (10):
>> Clangd :: did-change-configuration-params.test
>> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.CmdLineHash
>> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.DirectIncludesTest
>> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.IndexTwoFiles
>> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.NoCrashOnErrorFile
>> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.NoDotsInAbsPath
>> Clangd Unit Tests :: 
>> ./ClangdTests/BackgroundIndexTest.ShardStorageEmptyFile
>> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.ShardStorageLoad
>> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.ShardStorageTest
>> Clangd Unit Tests :: ./ClangdTests/BackgroundIndexTest.UncompilableFiles
>> 
>> 
>>> On Jul 18, 2019, at 9:25 AM, Kadir Cetinkaya via cfe-commits 
>>> mailto:cfe-commits@lists.llvm.org>> wrote:
>>> 
>>> Author: kadircet
>>> Date: Thu Jul 18 09:25:36 2019
>>> New Revision: 366458
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=366458&view=rev 
>>> 
>>> Log:
>>> [clangd] Refactor background-index shard loading
>>> 
>>> Reviewers: sammccall
>>> 
>>> Subscribers: mgorny, ilya-biryukov, MaskRay, jkorous, arphaman, cfe-commits
>>> 
>>> Tags: #clang
>>> 
>>> Differential Revision: https://reviews.llvm.org/D64712 
>>> 
>>> 
>>> Added:
>>>clang-tools-extra/trunk/clangd/index/BackgroundIndexLoader.cpp
>>>clang-tools-extra/trunk/clangd/index/BackgroundIndexLoader.h
>>> Modified:
>>>clang-tools-extra/trunk/clangd/CMakeLists.txt
>>>clang-tools-extra/trunk/clangd/index/Background.cpp
>>>clang-tools-extra/trunk/clangd/index/Background.h
>>>clang-tools-extra/trunk/clangd/index/BackgroundRebuild.cpp
>>>clang-tools-extra/trunk/clangd/index/BackgroundRebuild.h
>>>clang-tools-extra/trunk/clangd/unittests/BackgroundIndexTests.cpp
>>> 
>>> Modified: clang-tools-extra/trunk/clangd/CMakeLists.txt
>>> URL: 
>>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/CMakeLists.txt?rev=366458&r1=366457&r2=366458&view=diff
>>>  
>>> 
>>> ==
>>> --- clang-tools-extra/trunk/clangd/CMakeLists.txt (original)
>>> +++ clang-tools-extra/trunk/clangd/CMakeLists.txt Thu Jul 18 09:25:36 2019
>>> @@ -73,6 +73,7 @@ add_clang_library(clangDaemon
>>>   XRefs.cpp
>>> 
>>>   index/Background.cpp
>>> +  index/BackgroundIndexLoader.cpp
>>>   index/BackgroundIndexStorage.cpp
>>>   index/BackgroundQueue.cpp
>>>   index/BackgroundRebuild.cpp
>>> 
>>> Modified: clang-tools-extra/trunk/clangd/index/Background.cpp
>>> URL: 
>>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/index/Background.cpp?rev=366458&r1=366457&r2=366458&view=diff
>>>  
>>> 
>>> ==
>>> --- clang-tools-extra/trunk/clangd/index/Background.cpp (original)
>>> +++ clang-tools-extra/trunk/clangd/index/Background.cpp Thu Jul 18 09:25:36 
>>> 2019
>>> @@ -10,6 +10,7 @@
>>> #include "ClangdUnit.h"
>>> #include "Compiler.h"
>>> #include "Context.h"
>>> +#include "FSProvider.h"
>>> #include "Headers.h"
>>> #include "Logger.h"
>>> #include "Path.h"
>>> @@ -18,6 +19,7 @@
>>> #include "Threading.h"
>>> #include "Trace.h"
>>

Re: [PATCH] D66296: [BUNDLER]Improve the test, NFC.

2019-08-15 Thread Azhar Mohammed via cfe-commits
Seeing this now:

fatal error: error in backend: Global variable '' has an invalid section 
specifier '__CLANG_OFFLOAD_BUNDLE__host-x86_64-apple-darwin17.6.0': mach-o 
section specifier requires a segment whose length is between 1 and 16 
characters.
 <>clang-10: error: clang frontend command failed with exit code 70 (use -v to 
see invocation)
clang version 10.0.0 (http://labmaster3.local/git/llvm-project.git 
 
00782a4b68c71b9fb80b403b13f9ec67f07a87a4)
Target: x86_64-apple-darwin17.6.0
Thread model: posix
InstalledDir: 
/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/bin
clang-10: note: diagnostic msg: PLEASE submit a bug report to 
https://bugs.llvm.org/  and include the crash 
backtrace, preprocessed source, and associated run script.
clang-10: note: diagnostic msg: Error generating preprocessed source(s) - no 
preprocessable inputs.
error: incremental linking by external tool failed.

http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-incremental/1196/consoleFull#-2382751928254eaf0-7326-4999-85b0-388101f2d404
 



> On Aug 15, 2019, at 12:00 PM, Alexey Bataev via Phabricator via llvm-commits 
>  wrote:
> 
> ABataev added a comment.
> 
> In D66296#1632003 , @azharudd wrote:
> 
>> Looks like this is failing on Darwin:
>> 
>> http://green.lab.llvm.org/green/job/clang-stage1-cmake-RA-incremental/1193/consoleFull#-2382751928254eaf0-7326-4999-85b0-388101f2d404
>> 
>>  
>> /Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm-project/clang/test/Driver/clang-offload-bundler.c:120:15:
>>  error: CK-TEXTLL: expected string not found in input
>>  // CK-TEXTLL: @A = dso_local global i32 0
>>^
>>  
>> /Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Driver/Output/clang-offload-bundler.c.tmp.bundle3.ll:3:1:
>>  note: scanning from here
>>  ; ModuleID = 
>> '/Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/llvm-project/clang/test/Driver/clang-offload-bundler.c'
>>  ^
>>  
>> /Users/buildslave/jenkins/workspace/clang-stage1-cmake-RA-incremental/clang-build/tools/clang/test/Driver/Output/clang-offload-bundler.c.tmp.bundle3.ll:8:1:
>>  note: possible intended match here
>>  @A = global i32 0, align 4
>> 
> 
> 
> Committed the fix already
> 
> 
> Repository:
>  rL LLVM
> 
> CHANGES SINCE LAST ACTION
>  https://reviews.llvm.org/D66296/new/
> 
> https://reviews.llvm.org/D66296
> 
> 
> 
> ___
> llvm-commits mailing list
> llvm-comm...@lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


Re: [PATCH] D84703: [clang codegen][AArch64] Use llvm.aarch64.neon.fcvtzs/u where it's necessary

2020-08-03 Thread Azhar Mohammed via cfe-commits
Hey Eli,

Looks like this is causing the test-suite build to fail. Can you please take a 
look.

Refer to 
http://green.lab.llvm.org/green/job/test-suite-verify-machineinstrs-aarch64-O3/8035/consoleFull
 
.

FAILED: 
SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o
 
/Users/buildslave/jenkins/workspace/test-suite-verify-machineinstrs-aarch64-O3/test-suite-build/tools/timeit
 --summary 
SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o.time
 
/Users/buildslave/jenkins/workspace/test-suite-verify-machineinstrs-aarch64-O3/compiler/bin/clang
 -DNDEBUG -B 
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
 -Wno-unused-command-line-argument -mllvm -verify-machineinstrs -O3 -arch arm64 
-isysroot 
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS13.5.sdk
 -w -Werror=date-time -std=c99 -MD -MT 
SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o
 -MF 
SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o.d
 -o 
SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o
 -c 
/Users/buildslave/jenkins/workspace/test-suite-verify-machineinstrs-aarch64-O3/test-suite/SingleSource/UnitTests/Vector/AArch64/aarch64_neon_intrinsics.c
fatal error: error in backend: Cannot select: intrinsic 
%llvm.aarch64.neon.fcvtzs

Thanks
Azhar

> On Jul 30, 2020, at 3:42 PM, Eli Friedman via Phabricator via llvm-commits 
>  wrote:
> 
> This revision was landed with ongoing or failed builds.
> This revision was automatically updated to reflect the committed changes.
> Closed by commit rG8dfb5d767e70: [clang codegen][AArch64] Use 
> llvm.aarch64.neon.fcvtzs/u where it's necessary (authored by efriedma).
> 
> Changed prior to commit:
>  https://reviews.llvm.org/D84703?vs=281052&id=282069#toc
> 
> Repository:
>  rG LLVM Github Monorepo
> 
> CHANGES SINCE LAST ACTION
>  https://reviews.llvm.org/D84703/new/
> 
> https://reviews.llvm.org/D84703
> 
> Files:
>  clang/lib/CodeGen/CGBuiltin.cpp
>  clang/test/CodeGen/aarch64-neon-fcvt-intrinsics.c
>  clang/test/CodeGen/aarch64-neon-intrinsics.c
>  clang/test/CodeGen/aarch64-neon-misc.c
>  clang/test/CodeGen/aarch64-v8.2a-fp16-intrinsics.c
>  clang/test/CodeGen/aarch64-v8.2a-neon-intrinsics.c
> 
> ___
> llvm-commits mailing list
> llvm-comm...@lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits

___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits