Page MenuHomePhabricator

[HIPSPV][4/4] Add option to use llc to emit SPIR-V
Needs ReviewPublic

Authored by linjamaki on Sep 29 2021, 1:24 AM.

Details

Reviewers
Anastasia
Summary

Add two hidden options for emitting SPIR-V binary via LLC. The options are only meant for testing and development.

´--spirv-use-llc´ invokes in-tree llc. Not usable at the moment until the SPIR-V backend lands on LLVM.

´--spirv-use-llc=<path-to-llc>´ invokes llc tool given as path. Meant for trying out out-of-tree SPIR-V backend.

The HIPSPV tool chain is the only one responding to these options.

Diff Detail

Event Timeline

linjamaki created this revision.Sep 29 2021, 1:24 AM
linjamaki edited the summary of this revision. (Show Details)Oct 4 2021, 3:04 AM
linjamaki added a reviewer: Anastasia.
linjamaki updated this revision to Diff 376848.Oct 4 2021, 3:21 AM

Remove metavar from --spirv-use-llc.

linjamaki published this revision for review.Oct 4 2021, 3:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 4 2021, 3:22 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

Ok, is the idea to deprecate this flag once we switch to llc by default then?

Ok, is the idea to deprecate this flag once we switch to llc by default then?

The idea is to remove the flags if no one else needs them when the HIPSPV tool chain switches over to llc.