goldstein.w.n added a comment.

LGTM.

I'm not a maintainer to please wait a day or so to push so that others can 
review.



================
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.
 ///
----------------
interrupts too. Might as well add that if updating the comments.


================
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
----------------
out of curiosity, why "may" -> "can"?


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

https://reviews.llvm.org/D148653

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to