[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