fc-magnetite: src/g23m-fad/rlp/rlp

src/g23m-fad/rlp/rlp_kers.c: workaround for TI's compiler BUG

author	Mychaela Falconia <falcon@freecalypso.org>
date	Thu, 13 Oct 2016 03:14:28 +0000
parents	90eb61ecd093
children

comparison

equal deleted inserted replaced

-:d8197af7e058
+:338bf7edbd47
 +------------------------------------------------------------------------------
 */
 GLOBAL void sig_rcv_ker_ready_ind(void)
 {
+/*
+* FreeCalypso: when we tried to compile this code from TCS3.2
+* in our TCS211-mimicking Magnetite environment, we hit a BUG
+* in TI's compiler, or more precisely, the compiler version used
+* in TCS211: compiling this C module would fail as the compiler
+* was producing bad assembly which the asm step rejected.
+* Analysis of the generated asm showed that the switch table
+* and all of the switch case code was omitted, but the reference
+* to the switch table was still there, hence asm failing.
+*
+* The following four volatile char initialized but otherwise unused
+* automatic variables constitute the workaround we have added to
+* get this C module past the broken compiler.  This workaround was
+* inspired by the disassembly of the rlp_kers.obj binary object
+* from the TCS211 blob library: it appears that TI had applied a
+* very similar workaround on their end to get this code past the
+* same broken compiler.  The generated code we get is not exactly
+* the same as what's in the blob, so our recreation of TI's compiler
+* workaround is probably inexact, but it does the job of getting
+* this C module past the compiler.
+*/
+volatile char bug1 = 'A';
+volatile char bug2 = 'B';
+volatile char bug3 = 'C';
+volatile char bug4 = 0;
 TRACE_FUNCTION ("sig_rcv_ker_ready_ind()");
 switch (GET_STATE (KER))
 {

FreeCalypso > hg > fc-magnetite