[PATCH] D43094: AMDGPU: Enable PIC by default for amdgcn

Konstantin Zhuravlyov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 8 16:43:10 PST 2018


kzhuravl added a comment.

I thought we could override `isPICDefault` in AMDGPUToolChain instead of doing what is posted. But AMDGPUToolChain is only created for AMDHSA OS.

Do we want to enable PIC by default for all AMDGCNs? Or only for AMDHSA?


https://reviews.llvm.org/D43094





More information about the cfe-commits mailing list