[PATCH] D80316: [HardwareLoops] Intrinsic LangRef descriptions

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added inline comments.


================
Comment at: llvm/docs/LangRef.rst:14835
+specific instructions, or revert the hardware-loop to a normal loop if target
+specific restriction are not met and a hardware-loop can't be generated.
+
----------------
efriedma wrote:
> Thinking about it a bit more, I'm not sure we really want to promise these are stable.  They're sort of an internal implementation detail, and frontend and mid-level optimizations don't really have any business generating them.
> 
> Maybe explicitly state these may be modified in the future, and are not intended to be used outside the backend.
Agreed, and thanks, will add this. For exactly these reasons I could justify to myself that we hadn't document/exposed them before, but that is not entirely correct...


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

https://reviews.llvm.org/D80316