Merge #121

121: link.x: EXTERN(__INTERRUPTS) must come before PROVIDED r=therealprof a=japaric

or the user defined interrupts will be discarded when using fat LTO as seen
in #120

fixes #120

r? @rust-embedded/cortex-m (anyone)

Co-authored-by: Jorge Aparicio <[email protected]>

Author: bors[bot]

build.rs

@@ -24,14 +24,18 @@ fn main() {
     let mut f = if env::var_os("CARGO_FEATURE_DEVICE").is_some() {
         let mut f = File::create(out.join("link.x")).unwrap();
 
+        f.write_all(link_x).unwrap();
+
+        // *IMPORTANT*: The weak aliases (i.e. `PROVIDED`) must come *after* `EXTERN(__INTERRUPTS)`.
+        // Otherwise the linker will ignore user defined interrupts and always populate the table
+        // with the weak aliases.
         writeln!(
             f,
             r#"
 /* Provides weak aliases (cf. PROVIDED) for device specific interrupt handlers */
 /* This will usually be provided by a device crate generated using svd2rust (see `device.x`) */
 INCLUDE device.x"#
         ).unwrap();
-        f.write_all(link_x).unwrap();
         f
     } else {
         let mut f = File::create(out.join("link.x")).unwrap();