Skip to content

Pass -Xcompiler -fPIC as 2 separate arguments#251

Closed
malfet wants to merge 1 commit intopytorch:masterfrom
malfet:malfet/fix-PIC-argument-passing
Closed

Pass -Xcompiler -fPIC as 2 separate arguments#251
malfet wants to merge 1 commit intopytorch:masterfrom
malfet:malfet/fix-PIC-argument-passing

Conversation

@malfet
Copy link
Copy Markdown
Contributor

@malfet malfet commented Mar 17, 2020

nvcc from CUDA-10.1 fails to parse "-Xcompiler -fPIC" passed as single argument:

$ nvcc -V; nvcc "-Xcompiler -fPIC"
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2019 NVIDIA Corporation
Built on Wed_Apr_24_19:10:27_PDT_2019
Cuda compilation tools, release 10.1, V10.1.168
nvcc fatal   : Unknown option 'Xcompiler -fPIC'

Test Plan: CI

nvcc from CUDA-10.1 fails to parse "-Xcompiler -fPIC" passed as single argument:
```
$ nvcc -V; nvcc "-Xcompiler -fPIC"
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2019 NVIDIA Corporation
Built on Wed_Apr_24_19:10:27_PDT_2019
Cuda compilation tools, release 10.1, V10.1.168
nvcc fatal   : Unknown option 'Xcompiler -fPIC'
```

Test Plan: CI
Copy link
Copy Markdown

@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.

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

@facebook-github-bot
Copy link
Copy Markdown

@malfet merged this pull request in 113bde1.

malfet added a commit to malfet/pytorch that referenced this pull request Mar 18, 2020
Update gloo submodule to `113bde13035594cafdca247be953610b53026553` be compatible with separate compilation introduced by
pytorch/gloo#251

Test Plan: CI
facebook-github-bot pushed a commit to pytorch/pytorch that referenced this pull request Mar 19, 2020
Summary:
Update gloo submodule to `113bde13035594cafdca247be953610b53026553` be compatible with separate compilation introduced by
pytorch/gloo#251
Pull Request resolved: #34969

Test Plan: CI

Differential Revision: D20527163

Pulled By: malfet

fbshipit-source-id: 300d83d8fe95d57b8d740543efada3c56ac7b493
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Update gloo submodule to `113bde13035594cafdca247be953610b53026553` be compatible with separate compilation introduced by
pytorch/gloo#251
Pull Request resolved: pytorch#34969

Test Plan: CI

Differential Revision: D20527163

Pulled By: malfet

fbshipit-source-id: 300d83d8fe95d57b8d740543efada3c56ac7b493
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants