[PATCH] D52138: [Thumb] Add some integer abs testcases for different typesizes.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 4 03:47:21 PST 2019


RKSimon added a comment.
Herald added a project: LLVM.

What's happening with this? I'm keen to get D49837 <https://reviews.llvm.org/D49837> in soon



================
Comment at: test/CodeGen/Thumb/iabs.ll:3
+; RUN: llvm-objdump -disassemble -arch-name=thumb %t.o | FileCheck  --check-prefix=COUNT %s
+; RUN: llc < %s -mtriple=thumb-unknown-unknown | FileCheck %s
 
----------------
Instead of the counts, just running llc and checking the full codegen would be a lot cleaner.

I don't think the update_llc_test_checks script supports thumb triples but that'd be even easier.....


Repository:
  rL LLVM

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

https://reviews.llvm.org/D52138





More information about the llvm-commits mailing list