[PATCH] D132978: [IRTranslator] Using ZExt for extractelement indices.

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 12 01:10:49 PDT 2022


foad added inline comments.


================
Comment at: llvm/docs/LangRef.rst:9694-9695
 the position from which to extract the element. The index may be a
-variable of any integer type.
+variable of any integer type, and will be zero extended or truncated 
+depending on the target.
 
----------------
>From a semantics point of view, I think all we need to say is that the value is "treated as unsigned" or words to that effect. I don't think we should mention extending here (and definitely not truncating which sounds like it could change the value!).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132978



More information about the llvm-commits mailing list