Skip to content

Add LLVM patch to explicitly represent umin in SCEV #28403

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Aug 2, 2018
Merged

Add LLVM patch to explicitly represent umin in SCEV #28403

merged 2 commits into from
Aug 2, 2018

Conversation

@Keno
Copy link
Member

@Keno Keno commented Aug 2, 2018

Fixes #28379
Fixes #28388

@vchuravy If you would be so kind.

@ararslan ararslan requested a review from vchuravy August 2, 2018 02:21
vchuravy added a commit to staticfloat/LLVMBuilder that referenced this pull request Aug 2, 2018
@vchuravy
add patch for JuliaLang/julia#28403
4b33077
@ararslan ararslan mentioned this pull request Aug 2, 2018
Merged
7 tasks
@JeffBezanson JeffBezanson added the bugfix This change fixes an existing bug label Aug 2, 2018
@StefanKarpinski StefanKarpinski added this to the 0.7 milestone Aug 2, 2018
@vchuravy
Copy link
Member

vchuravy commented Aug 2, 2018

CI failure is:

Got exception Base.IOError("UDP send failed: address not available (EADDRNOTAVAIL)", -99) outside of a @test
  IOError: UDP send failed: address not available (EADDRNOTAVAIL)
  Stacktrace:
   [1]

@Keno do you want to merge as is or do you want to add the Polly fix first

@Keno
Copy link
Member Author

Keno commented Aug 2, 2018

Merge as is, we can figure out polly later.

@Keno Keno merged commit 463513f into master Aug 2, 2018
@vchuravy vchuravy deleted the kf/uminscev branch August 2, 2018 15:39
@Keno Keno mentioned this pull request Aug 4, 2018
Closed
KristofferC pushed a commit that referenced this pull request Feb 11, 2019
@Keno @KristofferC
Add LLVM patch to explicitly represent umin in SCEV (#28403)
f2c445b 
* Add LLVM patch to explicitly represent umin in SCEV

Fixes #28379
Fixes #28388

* bump LLVM BB_REL 6.0.0-6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vchuravy vchuravy Awaiting requested review from vchuravy

Assignees

No one assigned

Labels

bugfix This change fixes an existing bug

Projects

None yet

Milestone

0.7

Development

Successfully merging this pull request may close these issues.

5 participants

@Keno @vchuravy @StefanKarpinski @JeffBezanson