Skip to content
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.

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

[Cherry pick] LLVM BC support #14086

Closed

Conversation

omarzl
Copy link
Contributor

@omarzl omarzl commented Oct 6, 2021

Cherry pick request from #13889


Using rules_swift it is possible to generate LLVM IR BC files using swift.emit_bc feature, some days ago a refactor to use create_compilation_outputs from cc_common was merged, it is giving us the following error since the bc extension is missing, this pr fixes it

Error in create_compilation_outputs: 'Somefile.swift.bc' has wrong extension. The list of possible extensions for 'objects' is: .o,.obj,.pic.o,.opb

Using [rules_swift](https://github.com/bazelbuild/rules_swift) it is possible to generate LLVM IR BC files using `swift.emit_bc` feature, some days ago a [refactor](bazelbuild/rules_swift@8cf3635#diff-535d0dbea9614c6905a30dfdb0e708df3b16fc68f8930d6aba519342509dc657R1948) to use `create_compilation_outputs` from `cc_common` was merged, it is giving us the following error since the bc extension is missing, this pr fixes it

```
Error in create_compilation_outputs: 'Somefile.swift.bc' has wrong extension. The list of possible extensions for 'objects' is: .o,.obj,.pic.o,.opb
```

Closes bazelbuild#13889.

PiperOrigin-RevId: 392889724
(cherry picked from commit f53c389)
@omarzl omarzl requested a review from lberki as a code owner October 6, 2021 03:38
@google-cla google-cla bot added the cla: yes label Oct 6, 2021
@lberki lberki requested review from oquenchil and removed request for lberki October 7, 2021 08:45
@thii
Copy link
Member

thii commented Oct 8, 2021

Since 4.2.1 has been released, I think you have to request a new patch release 4.2.2 to the release manager #13558

@omarzl omarzl mentioned this pull request Oct 8, 2021
9 tasks
@omarzl omarzl closed this Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants