[PATCH] D60291: [llvm] Add isa_and_nonnull

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 4 16:27:15 PDT 2019


hubert.reinterpretcast added inline comments.


================
Comment at: llvm/docs/ProgrammersManual.rst:167
 
+``isa_and_nonnull<>``:
+  The ``isa_and_nonnull<>`` operator works just like the ``isa<>`` operator,
----------------
The name seems backwards. What does it mean to check "isa" on null? That question would not arise if the name was `nonnull_and_isa`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60291





More information about the llvm-commits mailing list