[PATCH] D148653: [Header][doc] Add/revise MONITOR/MWAIT[X] descriptions

Paul Robinson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 19 09:59:35 PDT 2023


probinson added inline comments.


================
Comment at: clang/lib/Headers/pmmintrin.h:278
 ///    the monitor event pending state. Data stored in the monitored address
 ///    range causes the processor to exit the pending state.
 ///
----------------
goldstein.w.n wrote:
> interrupts too. Might as well add that if updating the comments.
Ah, sorry missed this comment before I pushed. Updated in rG12426441



================
Comment at: clang/lib/Headers/pmmintrin.h:291
 /// \param __hints
-///    Optional hints for the monitoring state, which may vary by processor.
+///    Optional hints for the monitoring state, which can vary by processor.
 static __inline__ void __DEFAULT_FN_ATTRS
----------------
goldstein.w.n wrote:
> out of curiosity, why "may" -> "can"?
That was on the advice of my tech writer.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148653/new/

https://reviews.llvm.org/D148653



More information about the cfe-commits mailing list