[PATCH] D22793: IR: Introduce inbounds attribute on getelementptr indices.
Peter Collingbourne via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 26 15:08:45 PDT 2016
pcc added a comment.
> I think you need to update andIRFlags.
Because this attribute is not currently supported on instructions, we don't need to do that yet.
================
Comment at: include/llvm/IR/Operator.h:384-387
@@ -382,1 +383,6 @@
+ /// none.
+ Optional<unsigned> getInBoundsIndex() const {
+ if (SubclassOptionalData >> 1 == 0) return None;
+ return (SubclassOptionalData >> 1) - 1;
+ }
----------------
majnemer wrote:
> How will this play with `intersectOptionalDataWith`?
We discussed this on IRC and concluded that we can remove that function (D22830).
https://reviews.llvm.org/D22793
More information about the llvm-commits
mailing list