tweak Target.h documentation

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Author: Tom Yang

lldb/include/lldb/Target/Target.h

@@ -640,7 +640,9 @@ class Target : public std::enable_shared_from_this<Target>,
   ///
   /// \param[in] notify
   ///     If notify is true, and the Module is new to this Target,
-  ///     Target::ModulesDidLoad will be called.
+  ///     Target::ModulesDidLoad will be called. See note in
+  ///     Target::ModulesDidLoad about thread-safety with
+  ///     Target::GetOrCreateModule.
   ///     If notify is false, it is assumed that the caller is adding
   ///     multiple Modules and will call ModulesDidLoad with the
   ///     full list at the end.
@@ -941,8 +943,8 @@ class Target : public std::enable_shared_from_this<Target>,
   ///   - TargetProperties::GetPreloadSymbols()
   ///   - TargetProperties::GetParallelModuleLoad()
   ///
-  /// Warning: if called in parallel with Target::GetOrCreateModule, this may
-  /// result in a ABBA deadlock situation.
+  /// Warning: if preloading is active and this is called in parallel with
+  /// Target::GetOrCreateModule, this may result in a ABBA deadlock situation.
   void ModulesDidLoad(ModuleList &module_list);
 
   void ModulesDidUnload(ModuleList &module_list, bool delete_locations);