[PATCH] D32530: [SVE][IR] Scalable Vector IR Type

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 22 16:26:04 PDT 2019


hfinkel added inline comments.


================
Comment at: docs/LangRef.rst:8222
+result in IR is a value of the appropriate type, which will be a poison
+value at runtime if the hardware vector length is exceeded.
 
----------------
I believe that all you need to say is:

  For a scalable vector, if the value of ``idx`` exceeds the runtime length of the vector, the result is a poison value.

This part about the "result in the IR" is confusing, and is just stating a basic consistency fact about LLVM's type system. I recommend removing that.


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

https://reviews.llvm.org/D32530





More information about the llvm-commits mailing list