[PATCH] D79064: [test-suite] Register result codes with lit.

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 17 13:29:39 PDT 2020


Meinersbur added a comment.

In D79064#2098947 <https://reviews.llvm.org/D79064#2098947>, @yln wrote:

> ResultCode has a self-registering constructor now, calling
>  `add_result_category()` is no longer necessary.  See `5c244115c98a`.


Given the change:

  -    def __init__(self, name, isFailure):
  +    def __init__(self, name, label, isFailure):

every program using the two-argument ResultCode constructor will fail now? I also see the `add_result_category` has been removed entirely, and llvm-test-suite has not received an update.


Repository:
  rT test-suite

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

https://reviews.llvm.org/D79064





More information about the llvm-commits mailing list