[all-commits] [llvm/llvm-project] e8021f: ADT: Remove the unused explicit `OptionalTest` fix...

Duncan P. N. Exon Smith via All-commits all-commits at lists.llvm.org
Tue Dec 8 15:25:27 PST 2020


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e8021f34e1bae8537aba3c834ff0a9a73ad93405
      https://github.com/llvm/llvm-project/commit/e8021f34e1bae8537aba3c834ff0a9a73ad93405
  Author: Duncan P. N. Exon Smith <dexonsmith at apple.com>
  Date:   2020-12-08 (Tue, 08 Dec 2020)

  Changed paths:
    M llvm/unittests/ADT/OptionalTest.cpp

  Log Message:
  -----------
  ADT: Remove the unused explicit `OptionalTest` fixture, NFC

`OptionalTest` was empty; drop it and switch all the tests to use the
shorter `TEST` instead of `TEST_F`.

Differential Revision: https://reviews.llvm.org/D92675


  Commit: caab41c08e36872d91dddd4a05a5d98948802ac7
      https://github.com/llvm/llvm-project/commit/caab41c08e36872d91dddd4a05a5d98948802ac7
  Author: Duncan P. N. Exon Smith <dexonsmith at apple.com>
  Date:   2020-12-08 (Tue, 08 Dec 2020)

  Changed paths:
    M llvm/include/llvm/ADT/Optional.h
    M llvm/unittests/ADT/OptionalTest.cpp

  Log Message:
  -----------
  ADT: Add hash_value overload for Optional

Add a `hash_value` for Optional so that other data structures with
optional fields can easily hash them. I have a use for this in an
upcoming patch.

Differential Revision: https://reviews.llvm.org/D92676


Compare: https://github.com/llvm/llvm-project/compare/f0e89e69d6d7...caab41c08e36


More information about the All-commits mailing list