[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