[PATCH] D48176: [AMDGPU] Refactor HSAMetadataStream::emitKernel (NFC)

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added inline comments.


================
Comment at: llvm/trunk/lib/Target/AMDGPU/MCTargetDesc/AMDGPUHSAMetadataStreamer.h:31
 class Module;
+class SIProgramInfo;
 class Type;
----------------
I see some build errors. Seems to be due to this commit:
```
[1276/3790] Building CXX object lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUHSAMetadataStreamer.cpp.o
FAILED: lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUHSAMetadataStreamer.cpp.o 
/app/clang/3.6/bin/clang++  -march=corei7  -DGTEST_HAS_RTTI=0 -D_DEBUG -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Ilib/Target/AMDGPU -I../lib/Target/AMDGPU -I/usr/include/libxml2 -Iinclude -I../include -fPIC -fvisibility-inlines-hidden -Werror -Werror=date-time -std=c++11 -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-long-long -Wcovered-switch-default -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wstring-conversion -fdiagnostics-color -ffunction-sections -fdata-sections -O3    -UNDEBUG  -fno-exceptions -fno-rtti -MMD -MT lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUHSAMetadataStreamer.cpp.o -MF lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUHSAMetadataStreamer.cpp.o.d -o lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUHSAMetadataStreamer.cpp.o -c ../lib/Target/AMDGPU/AMDGPUHSAMetadataStreamer.cpp
In file included from ../lib/Target/AMDGPU/AMDGPUHSAMetadataStreamer.cpp:20:
../lib/Target/AMDGPU/SIProgramInfo.h:22:1: error: 'SIProgramInfo' defined as a struct here but previously declared as a class [-Werror,-Wmismatched-tags]
struct SIProgramInfo {
^
../lib/Target/AMDGPU/AMDGPUHSAMetadataStreamer.h:31:1: note: did you mean struct here?
class SIProgramInfo;
^~~~~
struct
1 error generated.
[1277/3790] Building CXX object lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUFrameLowering.cpp.o
[1278/3790] Building CXX object lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUAsmPrinter.cpp.o
FAILED: lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUAsmPrinter.cpp.o 
/app/clang/3.6/bin/clang++  -march=corei7  -DGTEST_HAS_RTTI=0 -D_DEBUG -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Ilib/Target/AMDGPU -I../lib/Target/AMDGPU -I/usr/include/libxml2 -Iinclude -I../include -fPIC -fvisibility-inlines-hidden -Werror -Werror=date-time -std=c++11 -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-long-long -Wcovered-switch-default -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wstring-conversion -fdiagnostics-color -ffunction-sections -fdata-sections -O3    -UNDEBUG  -fno-exceptions -fno-rtti -MMD -MT lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUAsmPrinter.cpp.o -MF lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUAsmPrinter.cpp.o.d -o lib/Target/AMDGPU/CMakeFiles/LLVMAMDGPUCodeGen.dir/AMDGPUAsmPrinter.cpp.o -c ../lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp
In file included from ../lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp:19:
In file included from ../lib/Target/AMDGPU/AMDGPUAsmPrinter.h:21:
../lib/Target/AMDGPU/SIProgramInfo.h:22:1: error: 'SIProgramInfo' defined as a struct here but previously declared as a class [-Werror,-Wmismatched-tags]
struct SIProgramInfo {
^
../lib/Target/AMDGPU/AMDGPUHSAMetadataStreamer.h:31:1: note: did you mean struct here?
class SIProgramInfo;
^~~~~
struct
1 error generated.
```


Repository:
  rL LLVM

https://reviews.llvm.org/D48176