[PATCH] D130089: update-test-checks: safely handle tests with #if's

Nicolai Hähnle via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 20 02:25:23 PDT 2022


nhaehnle added inline comments.


================
Comment at: llvm/test/tools/UpdateTestChecks/update_llc_test_checks/Inputs/arm_function_name.ll:2
 ; Check that we accept functions with '$' in the name.
 ; TODO: This is not handled correcly on 32bit ARM and needs to be fixed.
 
----------------
arichardson wrote:
> Remove this TODO?
Ok, will do before I commit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130089



More information about the cfe-commits mailing list