[PATCH] D49650: Targets/AMDGPU: Don't set fp32-denormals feature for r600

2018-07-27 Thread Jan Vesely via Phabricator via cfe-commits
jvesely abandoned this revision. jvesely added a comment. In https://reviews.llvm.org/D49650#1177323, @jvesely wrote: > In https://reviews.llvm.org/D49650#1176336, @arsenm wrote: > > > In https://reviews.llvm.org/D49650#1175461, @jvesely wrote: > > > > > In https://reviews.llvm.org/D49650#1175438

[PATCH] D49650: Targets/AMDGPU: Don't set fp32-denormals feature for r600

2018-07-26 Thread Jan Vesely via Phabricator via cfe-commits
jvesely added a comment. In https://reviews.llvm.org/D49650#1176336, @arsenm wrote: > In https://reviews.llvm.org/D49650#1175461, @jvesely wrote: > > > In https://reviews.llvm.org/D49650#1175438, @arsenm wrote: > > > > > According to cayman manual, these registers do exist so we should > > > pro

[PATCH] D49650: Targets/AMDGPU: Don't set fp32-denormals feature for r600

2018-07-26 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm added a comment. In https://reviews.llvm.org/D49650#1175461, @jvesely wrote: > In https://reviews.llvm.org/D49650#1175438, @arsenm wrote: > > > According to cayman manual, these registers do exist so we should probably > > just make the feature accepted on r600 as well > > > sure, that's

[PATCH] D49650: Targets/AMDGPU: Don't set fp32-denormals feature for r600

2018-07-25 Thread Jan Vesely via Phabricator via cfe-commits
jvesely added a comment. In https://reviews.llvm.org/D49650#1175438, @arsenm wrote: > According to cayman manual, these registers do exist so we should probably > just make the feature accepted on r600 as well sure, that's the way it was before r335942. I assumed the removal was intentional.

[PATCH] D49650: Targets/AMDGPU: Don't set fp32-denormals feature for r600

2018-07-25 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm added a comment. According to cayman manual, these registers do exist so we should probably just make the feature accepted on r600 as well Repository: rC Clang https://reviews.llvm.org/D49650 ___ cfe-commits mailing list cfe-commits@lists

[PATCH] D49650: Targets/AMDGPU: Don't set fp32-denormals feature for r600

2018-07-22 Thread Jan Vesely via Phabricator via cfe-commits
jvesely created this revision. jvesely added reviewers: arsenm, tstellar. Herald added subscribers: t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng, kzhuravl. This feature was removed in r335942 Repository: rC Clang https://reviews.llvm.org/D49650 Files: lib/Basic/Targets/AMDGPU.cpp Index