[mythtv-commits] Ticket #10762: Patch for compile with clang in greedyh.asm

[MythTV]

#10762: Patch for compile with clang in greedyh.asm
-------------------------------------+-------------------------------------
 Reporter:  Gary Buhrmaster          |           Type:  Patch - Feature
  <gary.buhrmaster@…>                |       Priority:  minor
   Status:  new                      |      Component:  MythTV - Video
Milestone:  unknown                  |  Playback
  Version:  Master Head              |       Severity:  medium
 Keywords:                           |  Ticket locked:  0
-------------------------------------+-------------------------------------
 On Linux x86_64 platforms with clang, the compiler appears to optimize out
 variables that are used in the assembly code(s) in the file greedyh.asm,
 resulting in the inability to link the codes.  This patch adds in the
 __used__ attribute hint to those variables (as is done for other
 variables).

 Longer term, it might make sense to port the current xine sources, which
 appear to have different improvements to that code area, but that would be
 a larger effort since the calling parms have also changed.  This is a
 "minimal" fix.

 Since clang is not an officially supported compiler, I am creating this as
 a feature patch.

 Env: clang/llvm 3.1 with mythtv git master

 Thanks for your consideration.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/10762>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center