RTEMS | sparc: Remove redundant 'register' specifier from cpuimpl.h (!835)

Joel Sherrill (@joel) gitlab at rtems.org
Thu Nov 20 16:49:41 UTC 2025



Merge request https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/835 was reviewed by Joel Sherrill

--
  
Joel Sherrill started a new discussion on cpukit/score/cpu/sparc/include/rtems/score/cpuimpl.h: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/835#note_136157

 >  {
 > -   register uint32_t g7 __asm__( "g7" );
 > +    uint32_t g7 __asm__( "g7" );

This should only have two spaces in from of it.

--
  
Joel Sherrill started a new discussion on cpukit/score/cpu/sparc/include/rtems/score/cpuimpl.h: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/835#note_136158

 > +    uint32_t g7 __asm__( "g7" );
 >  
 >     g7 = context->g7;

It seems better to use an inline asm to set g7. See sparc.h for various sparc_set_xxx() examples.


-- 
View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/835
You're receiving this email because of your account on gitlab.rtems.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/bugs/attachments/20251120/cabe45d6/attachment.htm>


More information about the bugs mailing list