[PATCH] D62867: UpdateTestChecks: hexagon support

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 4 10:58:02 PDT 2019


lebedev.ri added a comment.

In D62867#1529598 <https://reviews.llvm.org/D62867#1529598>, @kparzysz wrote:

> What is this in preparation of?  Why are only these 3 tests affected?


Because only these 3 tests are affected in the patch i'm about to post.

And having the ability to autogenerate these check lines radically
simplifies the developer's ability to work with the diff, instead of
first having to write the code, and then write the check lines too.

> I guess I'm ok with this, but



> I'm hoping you're not planning to convert all Hexagon tests to use these autogenerated checks...

Correct, i'm most certainly NOT planning on doing that.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62867





More information about the llvm-commits mailing list