Just includes patch to add bulk of target, not tests or other prerequisite patches.
Details
Diff Detail
Event Timeline
| lib/Target/HSAIL/CMakeLists.txt | ||
|---|---|---|
| 20 | Please leave this out of the first version. Having two code paths in CodeGen in pretty bad, specially when one of them depends on an external library. | |
| include/llvm/Support/ELF.h | ||
|---|---|---|
| 237 | I don't see these in http://www.sco.com/developers/gabi/latest/ch4.eheader.html. | |
| lib/Target/HSAIL/CMakeLists.txt | ||
| 51 | No commented out code please. | |
| lib/Target/HSAIL/HSAIL.h | ||
| 70 | Empty comments? | |
| 104 | Don't repeat names in comments. | |
| lib/Target/HSAIL/HSAILAlwaysInlinePass.cpp | ||
| 27 | please git-clang-format the patch. | |
| 59 | range loop? | |
| lib/Target/HSAIL/HSAILAsmPrinter.h | ||
| 34 | Start functions with lowercase letter when you can (or is this missing an override?) | |
| lib/Target/HSAIL/HSAILSection.cpp | ||
| 16 | Just don't create sections instead of having empty section switching. | |
Just some codingstyle issues I found while skimming over a few files
| lib/Target/HSAIL/AMDOpenCLKernenv.h | ||
|---|---|---|
| 11–13 | Needs three slashes. There are similar problems in many of the other files. | |
| 17–18 | Should be changed to LIB_TARGET_HSAIL_AMDOPENCLKERNELENV_H. There are similar issues in other headers. | |
| lib/Target/HSAIL/HSAIL.h | ||
| 26–44 | Better use static const unsigned or enum values so debuggers can pick those constants up as well. | |
| 218–222 | Could use doxygen comments here: GENERIC, ///< No target specific flavor ... | |
| lib/Target/HSAIL/HSAILAlwaysInlinePass.cpp | ||
| 25 | Would be nice to name the file like the class declared inside. | |
| test/CodeGen/HSAIL/128bit-kernel-args.ll | ||
| 2 | Many of the tests define new prefixes instead of just using "CHECK" should be unnecessary if you just test for 1 variant anyway. | |
| include/llvm/Support/ELF.h | ||
|---|---|---|
| 237 | This is being requested | |
| lib/Target/HSAIL/HSAILAlwaysInlinePass.cpp | ||
| 59 | I fixed this in the AMDGPU copy of this pass, so once http://reviews.llvm.org/D11155 is in I'll just remove this file. | |
| lib/Target/HSAIL/HSAILSection.cpp | ||
| 16 | These sections are created by the generic code in the AsmPrinter. This is exactly what NVPTX does currently to fix this. | |
| test/CodeGen/HSAIL/128bit-kernel-args.ll | ||
| 2 | People have expressed regret as having a default of CHECK for this before. The idea is someday it might be useful to add an HSAIL64 or something like that someday. | |
| lib/Target/HSAIL/HSAILISelDAGToDAG.cpp | ||
|---|---|---|
| 606–608 | This could probably be simplified a lot. The cases for ADD/OR looks unnecessary. It seems to mostly be to workaround the ADD -> OR combine. I don't think LDA needs to be handled here at all. I fixed it so that global addresses are handled during selection, so this shouldn't be seeing the custom node (which doesn't seem to be used anymore) | |
| lib/Target/HSAIL/MCTargetDesc/RawVectorOstream.h | ||
| 23–25 | I think I can completely remove this for now since it should only be used by the BRIGAsmPrinter | |
I don't see these in http://www.sco.com/developers/gabi/latest/ch4.eheader.html.