f use align() rather than repr(C), because we don't care about order

Author: TheBlueMatt

lightning/src/routing/router.rs

@@ -967,7 +967,7 @@ impl_writeable_tlv_based!(RouteHintHop, {
 });
 
 #[derive(Eq, PartialEq)]
-#[repr(C)] // Force our field ordering to keep the size 64 bytes
+#[repr(align(64))] // Force the size to 64 bytes
 struct RouteGraphNode {
 	node_id: NodeId,
 	score: u64,
@@ -994,7 +994,7 @@ impl cmp::PartialOrd for RouteGraphNode {
 	}
 }
 
-// While RouteGraphNode can be laid out as 56 bytes, performance appears to be improved
+// While RouteGraphNode can be laid out with fewer bytes, performance appears to be improved
 // substantially when it is laid out at exactly 64 bytes.
 //
 // Thus, we use `#[repr(C)]` on the struct to force a suboptimal layout and check that it stays 64