[PATCH] D59938: [DebugInfo] Make InstrRange into a class, NFC

David Stenberg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 10 01:34:21 PDT 2019


dstenb marked an inline comment as done.
dstenb added inline comments.


================
Comment at: include/llvm/CodeGen/DbgEntityHistoryCalculator.h:45
+
+    bool isEnded() const { return End; }
+
----------------
aprantl wrote:
> I think the proper terminology for this may be isOpen / is Closed? https://en.wikipedia.org/wiki/Interval_(mathematics)
Sorry, yes! I'll change it to "Closed" before landing this.


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

https://reviews.llvm.org/D59938





More information about the llvm-commits mailing list