[PATCH] D109390: [NFC] Cleanup off by one indexes in CallBase::dataOperandHasImpliedAttr()

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 7 14:49:51 PDT 2021


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm



================
Comment at: llvm/include/llvm/IR/InstrTypes.h:1655
 
-    // FIXME: Avoid these i - 1 calculations and update the API to use
-    // zero-based indices.
----------------
Looks like I wrote this in rGfb502d2f5e20e. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109390



More information about the llvm-commits mailing list