[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