[llvm] r182832 - Temporary fix to get rid of gcc warning.
Patrik Hagglund
patrik.h.hagglund at ericsson.com
Wed May 29 00:32:10 PDT 2013
Author: patha
Date: Wed May 29 02:32:08 2013
New Revision: 182832
URL: http://llvm.org/viewvc/llvm-project?rev=182832&view=rev
Log:
Temporary fix to get rid of gcc warning.
Modified:
llvm/trunk/lib/Target/R600/AMDGPUSubtarget.cpp
Modified: llvm/trunk/lib/Target/R600/AMDGPUSubtarget.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/R600/AMDGPUSubtarget.cpp?rev=182832&r1=182831&r2=182832&view=diff
==============================================================================
--- llvm/trunk/lib/Target/R600/AMDGPUSubtarget.cpp (original)
+++ llvm/trunk/lib/Target/R600/AMDGPUSubtarget.cpp Wed May 29 02:32:08 2013
@@ -37,7 +37,16 @@ AMDGPUSubtarget::AMDGPUSubtarget(StringR
ParseSubtargetFeatures(GPU, FS);
DevName = GPU;
Device = AMDGPUDeviceInfo::getDeviceFromName(DevName, this, Is64bit);
- TexVTXClauseSize = (Device->getGeneration() >= AMDGPUDeviceInfo::HD4XXX)?16:8;
+
+ // FIXME: The code in the comment below was the original code. But the
+ // condition is always true, generating a warning when compiled with
+ // gcc. Vincent Lejeune indicated in a mail to llvm-commits 2013-05-23 that he
+ // will look into this. The code 'TexVTXClauseSize = 16' is just a temporary
+ // equivalent replacement, to get rid of the compiler warning.
+
+ // TexVTXClauseSize = (Device->getGeneration() >= AMDGPUDeviceInfo::HD4XXX)?16:8;
+
+ TexVTXClauseSize = 16;
}
AMDGPUSubtarget::~AMDGPUSubtarget() {
More information about the llvm-commits
mailing list