<div dir="ltr">Reverted both with r340202</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 20, 2018 at 12:26 PM Samuel Pitoiset <<a href="mailto:samuel.pitoiset@gmail.com">samuel.pitoiset@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">You will need to revert r340172 too because it fixes compilation issues <br>
reported by buildbot. If you think it will be easier, feel free to <br>
revert both.<br>
<br>
On 8/20/18 9:22 PM, Vitaly Buka wrote:<br>
> Wound it be easier just revert and reland correct patch?<br>
> Also easier to backport it to other LLVM version as a single patch.<br>
> <br>
> On Mon, Aug 20, 2018 at 12:08 PM Samuel Pitoiset <br>
> <<a href="mailto:samuel.pitoiset@gmail.com" target="_blank">samuel.pitoiset@gmail.com</a> <mailto:<a href="mailto:samuel.pitoiset@gmail.com" target="_blank">samuel.pitoiset@gmail.com</a>>> wrote:<br>
> <br>
>  Â  Â This should be fixed by <a href="https://reviews.llvm.org/D50974" rel="noreferrer" target="_blank">https://reviews.llvm.org/D50974</a><br>
> <br>
>  Â  Â Sorry for the breakage and please don't revert it, I guess a fix should<br>
>  Â  Â be upstream soon.<br>
> <br>
>  Â  Â Thanks!<br>
> <br>
>  Â  Â On 8/20/18 8:36 PM, Vitaly Buka wrote:<br>
>  Â  Â  > Not sure how this suppose to work.<br>
>  Â  Â  > Now MAX_COMMON_ADDRESS allows to have AS1 and AS2 == 6<br>
>  Â  Â  > and (*ASAliasRules)[AS1][AS2].<br>
>  Â  Â  > but ASAliasRulesPrivIsZero and ASAliasRulesGenIsZero had only 6<br>
>  Â  Â elements<br>
>  Â  Â  ><br>
>  Â  Â  > If no one is looking, I am going to revert.<br>
>  Â  Â  ><br>
>  Â  Â  > On Mon, Aug 20, 2018 at 11:24 AM Vitaly Buka<br>
>  Â  Â <<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.com</a> <mailto:<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.com</a>><br>
>  Â  Â  > <mailto:<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.com</a> <mailto:<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.com</a>>>> wrote:<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â Similar for msan<br>
>  Â  Â  ><br>
>  Â  Â <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap-msan/builds/6784" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap-msan/builds/6784</a><br>
>  Â  Â  >  Â  Â and ubsan<br>
>  Â  Â  ><br>
>  Â  Â <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap-ubsan/builds/5631/steps/check-llvm%20ubsan/logs/stdio" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap-ubsan/builds/5631/steps/check-llvm%20ubsan/logs/stdio</a><br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â On Mon, Aug 20, 2018 at 11:22 AM Vitaly Buka<br>
>  Â  Â <<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.com</a> <mailto:<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.com</a>><br>
>  Â  Â  >  Â  Â <mailto:<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.com</a><br>
>  Â  Â <mailto:<a href="mailto:vitalybuka@google.com" target="_blank">vitalybuka@google.com</a>>>> wrote:<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â This looks caused by the patch<br>
>  Â  Â  ><br>
>  Â  Â <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap/builds/6883/steps/check-llvm%20asan/logs/stdio" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap/builds/6883/steps/check-llvm%20asan/logs/stdio</a><br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â FAIL: LLVM :: CodeGen/AMDGPU/amdgpu-alias-analysis.ll<br>
>  Â  Â (5325 of<br>
>  Â  Â  >  Â  Â  Â  Â 27229) ******************** TEST 'LLVM ::<br>
>  Â  Â  >  Â  Â  Â  Â CodeGen/AMDGPU/amdgpu-alias-analysis.ll' FAILED<br>
>  Â  Â  >  Â  Â  Â  Â ******************** Script: -- : 'RUN: at line 1';<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/bin/opt<br>
>  Â  Â -mtriple=amdgcn--<br>
>  Â  Â  >  Â  Â  Â  Â -O3 -aa-eval -print-all-alias-modref-info -disable-output <<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll<br>
>  Â  Â  >  Â  Â  Â  Â 2>&1 |<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/bin/FileCheck<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll<br>
>  Â  Â  >  Â  Â  Â  Â : 'RUN: at line 2';<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/bin/opt<br>
>  Â  Â -mtriple=r600--<br>
>  Â  Â  >  Â  Â  Â  Â -O3 -aa-eval -print-all-alias-modref-info -disable-output <<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll<br>
>  Â  Â  >  Â  Â  Â  Â 2>&1 |<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/bin/FileCheck<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll<br>
>  Â  Â  >  Â  Â  Â  Â -- Exit Code: 1 Command Output (stderr): --<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll:10:10:<br>
>  Â  Â  >  Â  Â  Â  Â error: CHECK: expected string not found in input ; CHECK:<br>
>  Â  Â  >  Â  Â  Â  Â NoAlias: i8 addrspace(1)* %p1, i8 addrspace(6)* %p ^<br>
>  Â  Â  >  Â  Â  Â  Â <stdin>:3:1: note: scanning from here Function:<br>
>  Â  Â  >  Â  Â  Â  Â test_32bit_addr_space: 2 pointers, 0 call sites ^<br>
>  Â  Â <stdin>:3:5:<br>
>  Â  Â  >  Â  Â  Â  Â note: possible intended match here Function:<br>
>  Â  Â  >  Â  Â  Â  Â test_32bit_addr_space: 2 pointers, 0 call sites ^ --<br>
>  Â  Â  >  Â  Â  Â  Â ******************** Testing: 0 .. 10.. 20 FAIL: LLVM ::<br>
>  Â  Â  >  Â  Â  Â  Â CodeGen/AMDGPU/widen-smrd-loads.ll (6391 of 27229)<br>
>  Â  Â  >  Â  Â  Â  Â ******************** TEST 'LLVM ::<br>
>  Â  Â  >  Â  Â  Â  Â CodeGen/AMDGPU/widen-smrd-loads.ll' FAILED<br>
>  Â  Â ********************<br>
>  Â  Â  >  Â  Â  Â  Â Script: -- : 'RUN: at line 1';<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/bin/llc<br>
>  Â  Â -amdgpu-codegenprepare-widen-constant-loads=0<br>
>  Â  Â  >  Â  Â  Â  Â -march=amdgcn -mcpu=tahiti -verify-machineinstrs <<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/widen-smrd-loads.ll<br>
>  Â  Â  >  Â  Â  Â  Â |<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/bin/FileCheck<br>
>  Â  Â  >  Â  Â  Â  Â -enable-var-scope -check-prefixes=GCN,SI<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/widen-smrd-loads.ll<br>
>  Â  Â  >  Â  Â  Â  Â : 'RUN: at line 2';<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/bin/llc<br>
>  Â  Â -amdgpu-codegenprepare-widen-constant-loads=0<br>
>  Â  Â  >  Â  Â  Â  Â -march=amdgcn -mcpu=tonga -verify-machineinstrs <<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/widen-smrd-loads.ll<br>
>  Â  Â  >  Â  Â  Â  Â |<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/bin/FileCheck<br>
>  Â  Â  >  Â  Â  Â  Â -enable-var-scope -check-prefixes=GCN,VI<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/widen-smrd-loads.ll<br>
>  Â  Â  >  Â  Â  Â  Â -- Exit Code: 1 Command Output (stderr): --<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â =================================================================<br>
>  Â  Â ==22327==ERROR:<br>
>  Â  Â  >  Â  Â  Â  Â AddressSanitizer: global-buffer-overflow on address<br>
>  Â  Â  >  Â  Â  Â  Â 0x000007134b68 at pc 0x0000012c05c8 bp 0x7fff2e2b6b40 sp<br>
>  Â  Â  >  Â  Â  Â  Â 0x7fff2e2b6b38 READ of size 1 at 0x000007134b68 thread T0 #0<br>
>  Â  Â  >  Â  Â  Â  Â 0x12c05c7 in getAliasResult<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp:99:10<br>
>  Â  Â  >  Â  Â  Â  Â #1 0x12c05c7 in<br>
>  Â  Â llvm::AMDGPUAAResult::alias(llvm::MemoryLocation<br>
>  Â  Â  >  Â  Â  Â  Â const&, llvm::MemoryLocation const&)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp:107<br>
>  Â  Â  >  Â  Â  Â  Â #2 0x3367dc1 in llvm::AAResults::alias(llvm::MemoryLocation<br>
>  Â  Â  >  Â  Â  Â  Â const&, llvm::MemoryLocation const&)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/AliasAnalysis.cpp:105:23<br>
>  Â  Â  >  Â  Â  Â  Â #3 0x3396a90 in alias<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/include/llvm/Analysis/AliasAnalysis.h:896:25<br>
>  Â  Â  >  Â  Â  Â  Â #4 0x3396a90 in llvm::BasicAAResult::aliasCheck(llvm::Value<br>
>  Â  Â  >  Â  Â  Â  Â const*, unsigned long, llvm::AAMDNodes, llvm::Value const*,<br>
>  Â  Â  >  Â  Â  Â  Â unsigned long, llvm::AAMDNodes, llvm::Value const*,<br>
>  Â  Â llvm::Value<br>
>  Â  Â  >  Â  Â  Â  Â const*)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/BasicAliasAnalysis.cpp:1779<br>
>  Â  Â  >  Â  Â  Â  Â #5 0x33951de in<br>
>  Â  Â llvm::BasicAAResult::alias(llvm::MemoryLocation<br>
>  Â  Â  >  Â  Â  Â  Â const&, llvm::MemoryLocation const&)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/BasicAliasAnalysis.cpp:780:23<br>
>  Â  Â  >  Â  Â  Â  Â #6 0x3367dc1 in llvm::AAResults::alias(llvm::MemoryLocation<br>
>  Â  Â  >  Â  Â  Â  Â const&, llvm::MemoryLocation const&)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/AliasAnalysis.cpp:105:23<br>
>  Â  Â  >  Â  Â  Â  Â #7 0x35d070c in<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â llvm::MemoryDependenceResults::getSimplePointerDependencyFrom(llvm::MemoryLocation<br>
>  Â  Â  >  Â  Â  Â  Â const&, bool,<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â llvm::ilist_iterator<llvm::ilist_detail::node_options<llvm::Instruction,<br>
>  Â  Â  >  Â  Â  Â  Â true, false, void>, false, false>, llvm::BasicBlock*,<br>
>  Â  Â  >  Â  Â  Â  Â llvm::Instruction*, unsigned int*)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/MemoryDependenceAnalysis.cpp:577:26<br>
>  Â  Â  >  Â  Â  Â  Â #8 0x35cfb45 in<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â llvm::MemoryDependenceResults::getSimplePointerDependencyFrom(llvm::MemoryLocation<br>
>  Â  Â  >  Â  Â  Â  Â const&, bool,<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â llvm::ilist_iterator<llvm::ilist_detail::node_options<llvm::Instruction,<br>
>  Â  Â  >  Â  Â  Â  Â true, false, void>, false, false>, llvm::BasicBlock*,<br>
>  Â  Â  >  Â  Â  Â  Â llvm::Instruction*, unsigned int*)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/MemoryDependenceAnalysis.cpp:448:12<br>
>  Â  Â  >  Â  Â  Â  Â #9 0x35ce759 in<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â llvm::MemoryDependenceResults::getPointerDependencyFrom(llvm::MemoryLocation<br>
>  Â  Â  >  Â  Â  Â  Â const&, bool,<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â llvm::ilist_iterator<llvm::ilist_detail::node_options<llvm::Instruction,<br>
>  Â  Â  >  Â  Â  Â  Â true, false, void>, false, false>, llvm::BasicBlock*,<br>
>  Â  Â  >  Â  Â  Â  Â llvm::Instruction*, unsigned int*)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/MemoryDependenceAnalysis.cpp:341:28<br>
>  Â  Â  >  Â  Â  Â  Â #10 0x12c95d4 in isClobberedInFunction<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Target/AMDGPU/AMDGPUAnnotateUniformValues.cpp:112:19<br>
>  Â  Â  >  Â  Â  Â  Â #11 0x12c95d4 in visitLoadInst<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Target/AMDGPU/AMDGPUAnnotateUniformValues.cpp:142<br>
>  Â  Â  >  Â  Â  Â  Â #12 0x12c95d4 in visitLoad<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/include/llvm/IR/Instruction.def:153<br>
>  Â  Â  >  Â  Â  Â  Â #13 0x12c95d4 in visit<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/include/llvm/IR/Instruction.def:153<br>
>  Â  Â  >  Â  Â  Â  Â #14 0x12c95d4 in<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â visit<llvm::ilist_iterator<llvm::ilist_detail::node_options<llvm::Instruction,<br>
>  Â  Â  >  Â  Â  Â  Â true, false, void>, false, false> ><br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/include/llvm/IR/InstVisitor.h:92<br>
>  Â  Â  >  Â  Â  Â  Â #15 0x12c95d4 in visit<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/include/llvm/IR/InstVisitor.h:107<br>
>  Â  Â  >  Â  Â  Â  Â #16 0x12c95d4 in<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â visit<llvm::ilist_iterator<llvm::ilist_detail::node_options<llvm::BasicBlock,<br>
>  Â  Â  >  Â  Â  Â  Â true, false, void>, false, false> ><br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/include/llvm/IR/InstVisitor.h:92<br>
>  Â  Â  >  Â  Â  Â  Â #17 0x12c95d4 in visit<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/include/llvm/IR/InstVisitor.h:103<br>
>  Â  Â  >  Â  Â  Â  Â #18 0x12c95d4 in (anonymous<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â namespace)::AMDGPUAnnotateUniformValues::runOnFunction(llvm::Function&)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Target/AMDGPU/AMDGPUAnnotateUniformValues.cpp:184<br>
>  Â  Â  >  Â  Â  Â  Â #19 0x47fea6d in<br>
>  Â  Â  >  Â  Â  Â  Â llvm::FPPassManager::runOnFunction(llvm::Function&)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/IR/LegacyPassManager.cpp:1586:27<br>
>  Â  Â  >  Â  Â  Â  Â #20 0x3464f8c in RunPassOnSCC<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/CallGraphSCCPass.cpp:166:25<br>
>  Â  Â  >  Â  Â  Â  Â #21 0x3464f8c in RunAllPassesOnSCC<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/CallGraphSCCPass.cpp:430<br>
>  Â  Â  >  Â  Â  Â  Â #22 0x3464f8c in (anonymous<br>
>  Â  Â  >  Â  Â  Â  Â namespace)::CGPassManager::runOnModule(llvm::Module&)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/CallGraphSCCPass.cpp:486<br>
>  Â  Â  >  Â  Â  Â  Â #23 0x47ffe7e in runOnModule<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/IR/LegacyPassManager.cpp:1669:27<br>
>  Â  Â  >  Â  Â  Â  Â #24 0x47ffe7e in<br>
>  Â  Â  >  Â  Â  Â  Â llvm::legacy::PassManagerImpl::run(llvm::Module&)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/IR/LegacyPassManager.cpp:1774<br>
>  Â  Â  >  Â  Â  Â  Â #25 0xa074af in compileModule(char**, llvm::LLVMContext&)<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/tools/llc/llc.cpp:597:8<br>
>  Â  Â  >  Â  Â  Â  Â #26 0xa00ae0 in main<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/tools/llc/llc.cpp:351:22<br>
>  Â  Â  >  Â  Â  Â  Â #27 0x7f33f66c22e0 in __libc_start_main<br>
>  Â  Â  >  Â  Â  Â  Â (/lib/x86_64-linux-gnu/libc.so.6+0x202e0) #28 0x8f4b99 in<br>
>  Â  Â _start<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â (/b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/bin/llc+0x8f4b99)<br>
>  Â  Â  >  Â  Â  Â  Â 0x000007134b68 is located 56 bytes to the left of global<br>
>  Â  Â  >  Â  Â  Â  Â variable '<string literal>' defined in<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â '/b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp:74:5'<br>
>  Â  Â  >  Â  Â  Â  Â (0x7134ba0) of size 129 '<string literal>' is ascii string<br>
>  Â  Â  >  Â  Â  Â  Â 'AS.GLOBAL_ADDRESS == 1 && AS.REGION_ADDRESS == 2 &&<br>
>  Â  Â  >  Â  Â  Â  Â AS.LOCAL_ADDRESS == 3 && AS.CONSTANT_ADDRESS == 4 &&<br>
>  Â  Â  >  Â  Â  Â  Â AS.PRIVATE_ADDRESS == 5' 0x000007134b68 is located 4 bytes to<br>
>  Â  Â  >  Â  Â  Â  Â the right of global variable 'ASAliasRulesGenIsZero'<br>
>  Â  Â defined in<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â '/b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp:63:28'<br>
>  Â  Â  >  Â  Â  Â  Â (0x7134b40) of size 36 SUMMARY: AddressSanitizer:<br>
>  Â  Â  >  Â  Â  Â  Â global-buffer-overflow<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp:99:10<br>
>  Â  Â  >  Â  Â  Â  Â in getAliasResult Shadow bytes around the buggy address:<br>
>  Â  Â  >  Â  Â  Â  Â 0x000080e1e910: 00 00 00 00 00 00 00 00 00 07 f9 f9 f9 f9<br>
>  Â  Â f9 f9<br>
>  Â  Â  >  Â  Â  Â  Â 0x000080e1e920: 00 00 00 00 00 00 00 01 f9 f9 f9 f9 00 00<br>
>  Â  Â 00 00<br>
>  Â  Â  >  Â  Â  Â  Â 0x000080e1e930: 00 00 00 00 00 00 00 00 00 00 00 00 00 00<br>
>  Â  Â 00 00<br>
>  Â  Â  >  Â  Â  Â  Â 0x000080e1e940: f9 f9 f9 f9 00 00 00 00 00 03 f9 f9 f9 f9<br>
>  Â  Â f9 f9<br>
>  Â  Â  >  Â  Â  Â  Â 0x000080e1e950: 00 00 00 00 00 00 04 f9 f9 f9 f9 f9 00 00<br>
>  Â  Â 00 00<br>
>  Â  Â  >  Â  Â  Â  Â =>0x000080e1e960: 00 00 00 00 00 00 00 00 00 00 00 00<br>
>  Â  Â 04[f9]f9<br>
>  Â  Â  >  Â  Â  Â  Â f9 0x000080e1e970: f9 f9 f9 f9 00 00 00 00 00 00 00 00 00<br>
>  Â  Â 00 00<br>
>  Â  Â  >  Â  Â  Â  Â 00 0x000080e1e980: 00 00 00 00 01 f9 f9 f9 f9 f9 f9 f9 00<br>
>  Â  Â 00 00<br>
>  Â  Â  >  Â  Â  Â  Â 00 0x000080e1e990: 00 00 00 00 00 00 00 01 f9 f9 f9 f9 00<br>
>  Â  Â 00 00<br>
>  Â  Â  >  Â  Â  Â  Â 00 0x000080e1e9a0: 00 00 00 00 00 00 01 f9 f9 f9 f9 f9 00<br>
>  Â  Â 00 00<br>
>  Â  Â  >  Â  Â  Â  Â 00 0x000080e1e9b0: 00 00 00 00 00 00 00 00 00 00 00 00 00<br>
>  Â  Â 00 00<br>
>  Â  Â  >  Â  Â  Â  Â 01 Shadow byte legend (one shadow byte represents 8<br>
>  Â  Â application<br>
>  Â  Â  >  Â  Â  Â  Â bytes): Addressable: 00 Partially addressable: 01 02 03<br>
>  Â  Â 04 05 06<br>
>  Â  Â  >  Â  Â  Â  Â 07 Heap left redzone: fa Freed heap region: fd Stack left<br>
>  Â  Â  >  Â  Â  Â  Â redzone: f1 Stack mid redzone: f2 Stack right redzone: f3<br>
>  Â  Â Stack<br>
>  Â  Â  >  Â  Â  Â  Â after return: f5 Stack use after scope: f8 Global redzone: f9<br>
>  Â  Â  >  Â  Â  Â  Â Global init order: f6 Poisoned by user: f7 Container<br>
>  Â  Â overflow:<br>
>  Â  Â  >  Â  Â  Â  Â fc Array cookie: ac Intra object redzone: bb ASan<br>
>  Â  Â internal: fe<br>
>  Â  Â  >  Â  Â  Â  Â Left alloca redzone: ca Right alloca redzone: cb Shadow<br>
>  Â  Â gap: cc<br>
>  Â  Â  >  Â  Â  Â  Â ==22327==ABORTING<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm/test/CodeGen/AMDGPU/widen-smrd-loads.ll:143:14:<br>
>  Â  Â  >  Â  Â  Â  Â error: GCN-LABEL: expected string not found in input ;<br>
>  Â  Â  >  Â  Â  Â  Â GCN-LABEL: {{^}}widen_i16_constant32_load: ^ <stdin>:518:37:<br>
>  Â  Â  >  Â  Â  Â  Â note: scanning from here<br>
>  Â  Â widen_i1_zext_to_i64_constant_load: ;<br>
>  Â  Â  >  Â  Â  Â  Â @widen_i1_zext_to_i64_constant_load ^ <stdin>:518:48: note:<br>
>  Â  Â  >  Â  Â  Â  Â possible intended match here<br>
>  Â  Â widen_i1_zext_to_i64_constant_load:<br>
>  Â  Â  >  Â  Â  Â  Â ; @widen_i1_zext_to_i64_constant_load ^ --<br>
>  Â  Â ********************<br>
>  Â  Â  >  Â  Â  Â  Â Testing: 0 .. 10.. 20.. 30.. 40.. 50.. 60.. 70.. 80.. 90..<br>
>  Â  Â  >  Â  Â  Â  Â Testing Time: 384.24s ******************** Failing Tests (2):<br>
>  Â  Â  >  Â  Â  Â  Â LLVM :: CodeGen/AMDGPU/amdgpu-alias-analysis.ll LLVM ::<br>
>  Â  Â  >  Â  Â  Â  Â CodeGen/AMDGPU/widen-smrd-loads.ll Expected Passes : 26393<br>
>  Â  Â  >  Â  Â  Â  Â Expected Failures : 148 Unsupported Tests : 686 Unexpected<br>
>  Â  Â  >  Â  Â  Â  Â Failures: 2 FAILED: test/CMakeFiles/check-llvm cd<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/test<br>
>  Â  Â  >  Â  Â  Â  Â && /usr/bin/python2.7<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/./bin/llvm-lit<br>
>  Â  Â  >  Â  Â  Â  Â -sv<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â /b/sanitizer-x86_64-linux-bootstrap/build/llvm_build_asan/test<br>
>  Â  Â  >  Â  Â  Â  Â ninja: build stopped: subcommand failed. + echo<br>
>  Â  Â  >  Â  Â  Â  Â @@@STEP_FAILURE@@@ + echo @@@BUILD_STEP check-clang asan@@@<br>
>  Â  Â  >  Â  Â  Â  Â @@@STEP_FAILURE@@@<br>
>  Â  Â  >  Â  Â  Â <br>
>  Â  Â  Â --------------------------------------------------------------------------------<br>
>  Â  Â  >  Â  Â  Â  Â started: Mon Aug 20 06:50:07 2018 ended: Mon Aug 20 07:15:07<br>
>  Â  Â  >  Â  Â  Â  Â 2018 duration: 24 mins, 59 secs<br>
>  Â  Â  ><br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â On Mon, Aug 20, 2018 at 6:19 AM Samuel Pitoiset via<br>
>  Â  Â llvm-commits<br>
>  Â  Â  >  Â  Â  Â  Â <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
>  Â  Â <mailto:<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>><br>
>  Â  Â  >  Â  Â  Â  Â <mailto:<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
>  Â  Â <mailto:<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>>>> wrote:<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Author: hakzsam<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Date: Mon Aug 20 06:18:59 2018<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â New Revision: 340171<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â URL:<br>
>  Â  Â <a href="http://llvm.org/viewvc/llvm-project?rev=340171&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=340171&view=rev</a><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Log:<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â AMDGPU: bump AS.MAX_COMMON_ADDRESS to 6 since 32-bit<br>
>  Â  Â addr space<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â 32-bit constant address space is declared as 6, so the<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â maximum number of address spaces is 6, not 5.<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Fixes "LLVM ERROR: Pointer address space out of range".<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â v3: use static_assert()<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â v2: add a very simple test for 32-bit addr space<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Bugzilla:<br>
>  Â  Â <a href="https://bugs.freedesktop.org/show_bug.cgi?id=106630" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=106630</a><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Signed-off-by: Samuel Pitoiset<br>
>  Â  Â <<a href="mailto:samuel.pitoiset@gmail.com" target="_blank">samuel.pitoiset@gmail.com</a> <mailto:<a href="mailto:samuel.pitoiset@gmail.com" target="_blank">samuel.pitoiset@gmail.com</a>><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â <mailto:<a href="mailto:samuel.pitoiset@gmail.com" target="_blank">samuel.pitoiset@gmail.com</a><br>
>  Â  Â <mailto:<a href="mailto:samuel.pitoiset@gmail.com" target="_blank">samuel.pitoiset@gmail.com</a>>>><br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Modified:<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â  llvm/trunk/lib/Target/AMDGPU/AMDGPU.h<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â  llvm/trunk/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â <br>
>  Â  Â llvm/trunk/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Modified: llvm/trunk/lib/Target/AMDGPU/AMDGPU.h<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â URL:<br>
>  Â  Â  ><br>
>  Â  Â <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AMDGPU/AMDGPU.h?rev=340171&r1=340170&r2=340171&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AMDGPU/AMDGPU.h?rev=340171&r1=340170&r2=340171&view=diff</a><br>
>  Â  Â  >  Â  Â  Â  Â  Â <br>
>  Â  Â  Â ==============================================================================<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â --- llvm/trunk/lib/Target/AMDGPU/AMDGPU.h (original)<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +++ llvm/trunk/lib/Target/AMDGPU/AMDGPU.h Mon Aug 20<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â 06:18:59 2018<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â @@ -229,7 +229,7 @@ struct AMDGPUAS {<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â enum : unsigned {<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â  Â // The maximum value for flat, generic, local,<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â private, constant and region.<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â -  Â  MAX_COMMON_ADDRESS = 5,<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +  Â  MAX_COMMON_ADDRESS = 6,<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â  Â GLOBAL_ADDRESS = 1,  Â ///< Address space for global<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â memory (RAT0, VTX0).<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â  Â CONSTANT_ADDRESS = 4, ///< Address space for<br>
>  Â  Â constant<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â memory (VTX2)<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Modified:<br>
>  Â  Â llvm/trunk/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â URL:<br>
>  Â  Â  ><br>
>  Â  Â <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp?rev=340171&r1=340170&r2=340171&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp?rev=340171&r1=340170&r2=340171&view=diff</a><br>
>  Â  Â  >  Â  Â  Â  Â  Â <br>
>  Â  Â  Â ==============================================================================<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â --- llvm/trunk/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â (original)<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +++<br>
>  Â  Â llvm/trunk/lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp Mon<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Aug 20 06:18:59 2018<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â @@ -69,7 +69,7 @@<br>
>  Â  Â AMDGPUAAResult::ASAliasRulesTy::ASAliasR<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â /* Region  Â */ {MayAlias, NoAlias , NoAlias ,<br>
>  Â  Â NoAlias,<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â MayAlias, NoAlias},<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â /* Private  */ {MayAlias, NoAlias , NoAlias ,<br>
>  Â  Â NoAlias ,<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â NoAlias , MayAlias}<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â };<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â -  assert(AS.MAX_COMMON_ADDRESS <= 5);<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +  static_assert(AS.MAX_COMMON_ADDRESS <= 6, "Addr<br>
>  Â  Â space out<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â of range");<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â if (AS.FLAT_ADDRESS == 0) {<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â  Â assert(AS.GLOBAL_ADDRESS  Â == 1 &&<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  Â  AS.REGION_ADDRESS  Â == 2 &&<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Modified:<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â llvm/trunk/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â URL:<br>
>  Â  Â  ><br>
>  Â  Â <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll?rev=340171&r1=340170&r2=340171&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll?rev=340171&r1=340170&r2=340171&view=diff</a><br>
>  Â  Â  >  Â  Â  Â  Â  Â <br>
>  Â  Â  Â ==============================================================================<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â ---<br>
>  Â  Â llvm/trunk/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â (original)<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +++<br>
>  Â  Â llvm/trunk/test/CodeGen/AMDGPU/amdgpu-alias-analysis.ll<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â Mon Aug 20 06:18:59 2018<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â @@ -7,3 +7,9 @@ define void @test(i8 addrspace(5)* %p, i<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â  Â ret void<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â  Â }<br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +; CHECK: NoAlias:  Â  Â  i8 addrspace(1)* %p1, i8<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â addrspace(6)* %p<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +define void @test_32bit_addr_space(i8 addrspace(6)*<br>
>  Â  Â %p, i8<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â addrspace(1)* %p1) {<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +  ret void<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +}<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â +<br>
>  Â  Â  ><br>
>  Â  Â  ><br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â _______________________________________________<br>
>  Â  Â  >  Â  Â  Â  Â  Â  Â llvm-commits mailing list<br>
>  Â  Â  > <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a> <mailto:<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>><br>
>  Â  Â <mailto:<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
>  Â  Â <mailto:<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>>><br>
>  Â  Â  > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
>  Â  Â  ><br>
> <br>
</blockquote></div>