Review comments

Author: JamieCunliffe

compiler/rustc_middle/src/ty/context.rs

@@ -2088,6 +2088,8 @@ impl<'tcx> TyCtxt<'tcx> {
         self.sess.dcx()
     }
 
+    /// Checks to see if the caller (`body_features`) has all the features required by the callee
+    /// (`callee_features`).
     pub fn is_target_feature_call_safe(
         self,
         callee_features: &[TargetFeature],

tests/codegen-llvm/inline-always-callsite.rs

@@ -34,4 +34,8 @@ pub fn inherits_from_global() -> i32 {
     }
 }
 
+// Attribute #3 requires the alwaysinline attribute, the alwaysinline attribute is not emitted on a
+// function definition when target features are present, rather it will be moved onto the function
+// call, if the features match up.
+//
 // CHECK: attributes #3 = { alwaysinline nounwind }