Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@swernli
Copy link
Collaborator

@swernli swernli commented Jun 29, 2021

Since the packaged Fullstate Simulator always uses the Windows style dll name and local development (or xplat pipelines) need to handle the platform specific library name, we will imitate the loading patterns from .NET and try both when not on Windows.

@swernli swernli requested a review from ScottCarda-MS June 29, 2021 21:48
Since the packaged Fullstate Simulator always uses the Windows style dll name and local development (or xplat pipelines) need to handle the platform specific library name, we will imitiate the loading patterns from .NET and try both when not on Windows.
@swernli swernli force-pushed the swernli/qir-sim-dll branch from a83aba5 to f02bd35 Compare June 29, 2021 23:16
@swernli swernli changed the title Remove sim platform specific names Fix QIR Simulator dll load Jun 29, 2021
@swernli swernli requested a review from ScottCarda-MS June 29, 2021 23:17
@ScottCarda-MS ScottCarda-MS merged commit 54a4d6b into feature/azure-quantum-simulator Jun 30, 2021
cesarzc added a commit that referenced this pull request Jul 8, 2021
* Setup controller project (#597)
* Implement dependencies and tests (#626)
* Refactor controller into library (#642)
* Move QIR driver generator (#660)
* Ignore line breaks for testing purposes. (#671)
* Microsoft.Quantum.Qir.Tools package (#670)
* QIR Build From Q# DLL (#658)
* Add test for test.submitter.noop with Q#
* Command-Line Compiler (#681)
* QIR Command Line Tool - Automatically Use Required Resources (#704)
* QIR Command Line Tool - Packaging Fixes (#723)
* Update QIR submitter factory regex. (#728)`
Co-authored-by: Bettina Heim <beheim@microsoft.com>
* Fix QIR Simulator dll load (#760)
* Fix QIR Simulator dll load
* Fixing link (#762)
* Adding entry point operation loader (#766)
Co-authored-by: Zander Chocron <alchocro@microsoft.com>
Co-authored-by: Sarah Marshall <sarah.marshall@microsoft.com>
Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
Co-authored-by: Scott Carda <55811729+ScottCarda-MS@users.noreply.github.com>
Co-authored-by: Scott Carda <sccarda@microsoft.com>
Co-authored-by: Bettina Heim <beheim@microsoft.com>
Co-authored-by: bettinaheim <34236215+bettinaheim@users.noreply.github.com>
Co-authored-by: Stefan J. Wernli <swernli@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants