Update the documentation of CellSPU, in case it gets removed in 3.1.
authorKalle Raiskila <kalle.raiskila@nokia.com>
Sun, 29 Apr 2012 20:00:55 +0000 (20:00 +0000)
committerKalle Raiskila <kalle.raiskila@nokia.com>
Sun, 29 Apr 2012 20:00:55 +0000 (20:00 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@155797 91177308-0d34-0410-b5e6-96231b3b80d8

CREDITS.TXT
lib/Target/CellSPU/README.txt

index bf32a4c565bb560a634aacafc7cdadc04d912a6a..6b10a0de07928e74c13ab6e2876f98bdb0eae690 100644 (file)
@@ -342,6 +342,10 @@ W: http://vladimir_prus.blogspot.com
 E: ghost@cs.msu.su
 D: Made inst_iterator behave like a proper iterator, LowerConstantExprs pass
 
+N: Kalle Raiskila
+E: kalle.rasikila@nokia.com
+D: Some bugfixes to CellSPU
+
 N: Xerxes Ranby
 E: xerxes@zafena.se
 D: Cmake dependency chain and various bug fixes
index 3e7e0b68e8e623304e75d8837bcb4b28ebcfe5c5..3bce9609bfef5a18ed85aa539f1ed2ad6450f2ff 100644 (file)
@@ -37,6 +37,20 @@ to add 'spu' to configure's --enable-targets option, e.g.:
 ---------------------------------------------------------------------------
 
 TODO:
+* In commit r142152 vector legalization was set to element promotion per
+  default. This breaks half vectors (e.g. v2i32) badly as they get element
+  promoted to much slower types (v2i64).
+
+* Many CellSPU specific codegen tests only grep & count the number of 
+  instructions, not checking their place with FileCheck. There have also
+  been some commits that change the CellSPU checks, some of which might
+  have not been thoroughly scrutinized w.r.t. to the changes they cause in SPU
+  assembly. (especially since about the time of r142152)  
+
+* Some of the i64 math have huge tablegen rules, which sometime cause
+  tablegen to run out of memory. See e.g. bug 8850. i64 arithmetics 
+  should probably be done with libraries.
+
 * Create a machine pass for performing dual-pipeline scheduling specifically
   for CellSPU, and insert branch prediction instructions as needed.