Skip to content

Conversation

@asuhan
Copy link
Contributor

@asuhan asuhan commented Oct 29, 2020

Summary:

Adjust llvm_{codegen, jit}.cpp to support LLVM versions 8 through 12.

Test Plan:

test_tensorexpr --gtest_filter=TensorExprTest.LLVM*

@asuhan asuhan requested a review from apaszke as a code owner October 29, 2020 00:41
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Oct 29, 2020
@dr-ci
Copy link

dr-ci bot commented Oct 29, 2020

💊 CI failures summary and remediations

As of commit a66e26c (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 3 times.

Summary:

Adjust llvm_{codegen, jit}.cpp to support LLVM versions 8 through 12.

Test Plan: CI

Reviewers:

Subscribers:

Tasks:

Tags:
@asuhan asuhan force-pushed the asuhan/llvm_upgrade branch from ae9d071 to a66e26c Compare October 29, 2020 03:03
@asuhan asuhan changed the title Support LLVM versions 8 through 12 [TensorExpr] Support LLVM versions 8 through 12 Oct 29, 2020
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@asuhan has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@asuhan
Copy link
Contributor Author

asuhan commented Nov 2, 2020

Thanks!

@facebook-github-bot
Copy link
Contributor

@asuhan merged this pull request in 4df7eef.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged oncall: jit Add this issue/PR to JIT oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants