[PATCH] D130076: [llvm/CodeGen] Enable the ExpandLargeDivRem pass for X86, Arm and AArch64

[Matthias Gehre via Phabricator via llvm-commits]

mgehre-amd added a comment.

In D130076#3772580 <https://reviews.llvm.org/D130076#3772580>, @vitalybuka wrote:

> Breaks https://lab.llvm.org/buildbot/#/builders/85/builds/10520
>
>   ******************** TEST 'LLVM :: Transforms/ExpandLargeDivRem/udiv129.ll' FAILED ********************
>   Script:
>   --
>   : 'RUN: at line 2';   /b/sanitizer-x86_64-linux-bootstrap-ubsan/build/llvm_build_ubsan/bin/opt -S -expand-large-div-rem < /b/sanitizer-x86_64-linux-bootstrap-ubsan/build/llvm-project/llvm/test/Transforms/ExpandLargeDivRem/udiv129.ll | /b/sanitizer-x86_64-linux-bootstrap-ubsan/build/llvm_build_ubsan/bin/FileCheck /b/sanitizer-x86_64-linux-bootstrap-ubsan/build/llvm-project/llvm/test/Transforms/ExpandLargeDivRem/udiv129.ll
>   --
>   Exit Code: 1
>   
>   Command Output (stderr):
>   --
>   /b/sanitizer-x86_64-linux-bootstrap-ubsan/build/llvm-project/llvm/test/Transforms/ExpandLargeDivRem/udiv129.ll:6:15: error: CHECK-NEXT: expected string not found in input
>   ; CHECK-NEXT: _udiv-special-cases:
>                 ^
>   <stdin>:5:19: note: scanning from here
>   define void @test(i129* %ptr, i129* %out) #0 {
>                     ^
>   <stdin>:7:9: note: possible intended match here
>    %res = udiv i129 %a, 3
>           ^
>   
>   Input file: <stdin>
>   Check file: /b/sanitizer-x86_64-linux-bootstrap-ubsan/build/llvm-project/llvm/test/Transforms/ExpandLargeDivRem/udiv129.ll
>   
>   -dump-input=help explains the following input dump.
>   
>   Input was:
>   <<<<<<
>             1: ; ModuleID = '<stdin>' 
>             2: source_filename = "<stdin>" 
>             3:  
>             4: ; Function Attrs: nounwind 
>             5: define void @test(i129* %ptr, i129* %out) #0 { 
>   next:6'0                       X~~~~~~~~~~~~~~~~~~~~~~~~~~~~ error: no match found
>             6:  %a = load i129, i129* %ptr, align 4 
>   next:6'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>             7:  %res = udiv i129 %a, 3 
>   next:6'0     ~~~~~~~~~~~~~~~~~~~~~~~~
>   next:6'1             ?                possible intended match
>             8:  store i129 %res, i129* %out, align 4 
>   next:6'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>             9:  ret void 
>   next:6'0     ~~~~~~~~~~
>            10: } 
>   next:6'0     ~~
>            11:  
>   next:6'0     ~
>            12: attributes #0 = { nounwind } 
>   next:6'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>   >>>>>>

Thanks for letting me know, I already pushed a fix.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130076/new/

https://reviews.llvm.org/D130076