[PATCH] D22793: IR: Introduce inbounds attribute on getelementptr indices.

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 26 11:50:40 PDT 2016


pcc added a comment.

I think my concern with a separate keyword was that it would essentially need to be a synonym of `inbounds` in order to be descriptive, and I generally prefer not to use synonyms to refer to different things, as that can be confusing. But maybe using the same keyword would be even more confusing, and perhaps the overriding concern here should be searchability in langref. With that considered, I think I'm ok with `inrange`.


https://reviews.llvm.org/D22793





More information about the llvm-commits mailing list