Fix runtime crash by ARMC6 build

toyowata · toyowata · commit f9242d0b5c8e · 2019-12-19T15:31:33.000+09:00
diff --git a/targets/TARGET_NXP/TARGET_LPC11UXX/device/cmsis_nvic.c b/targets/TARGET_NXP/TARGET_LPC11UXX/device/cmsis_nvic.c
@@ -60,7 +60,8 @@ void NVIC_SetVector(IRQn_Type IRQn, uint32_t vector) {
     
     // Copy and switch to dynamic vectors if first time called
     if((LPC_SYSCON->SYSMEMREMAP & 0x3) != 0x1) {     
-      uint32_t *old_vectors = (uint32_t *)0;         // FLASH vectors are at 0x0
+      // Add volatile qualifier to avoid armclang aggressive optimization
+      volatile uint32_t *old_vectors = (uint32_t *)0;         // FLASH vectors are at 0x0
       for(i = 0; i < NVIC_NUM_VECTORS; i++) {    
             vectors[i] = old_vectors[i];
         }

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,8 @@ void NVIC_SetVector(IRQn_Type IRQn, uint32_t vector) {`
`60`	`60`
`61`	`61`	`// Copy and switch to dynamic vectors if first time called`
`62`	`62`	`if((LPC_SYSCON->SYSMEMREMAP & 0x3) != 0x1) {`
`63`		`- uint32_t old_vectors = (uint32_t )0; // FLASH vectors are at 0x0`
	`63`	`+ // Add volatile qualifier to avoid armclang aggressive optimization`
	`64`	`+ volatile uint32_t old_vectors = (uint32_t )0; // FLASH vectors are at 0x0`
`64`	`65`	`for(i = 0; i < NVIC_NUM_VECTORS; i++) {`
`65`	`66`	`vectors[i] = old_vectors[i];`
`66`	`67`	`}`