[PATCH] D83004: [UpdateCCTestChecks] Include generated functions if asked

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 01:51:01 PDT 2020


arichardson added inline comments.


================
Comment at: llvm/utils/update_cc_test_checks.py:331
+            # are ordered by prefix instead of by function as in "normal"
+            # mode.
+            if '-emit-llvm' in clang_args:
----------------
jdoerfert wrote:
> This is all unfortunate but at least for OpenMP not easily changeable. Let's go with this.
How difficult would it be to only use this behaviour if the normal one fails? 

If we encounter a function that's not in the source code we just append the check lines for the generate function where we last added checks. And if we then see a function where the order doesn't match the source order fall back to just emitting all checks at the end of the file?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83004



More information about the llvm-commits mailing list