[PATCH] D46339: [GlobalISel][Legalizer] LegalizerInfo verifier: Follow Up

Roman Tereshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 11 13:18:06 PDT 2018


rtereshin added inline comments.


================
Comment at: test/CodeGen/AArch64/GlobalISel/legalize-inttoptr-xfail-1.mir:1
 # RUN: not llc -mtriple=aarch64-- -run-pass=legalizer -simplify-mir %s -o - 2>&1 | FileCheck %s
 # REQUIRES: asserts
----------------
qcolombet wrote:
> Out-of-curiosity, why is the simplify-mir option required here?
It's not required, but I find it useful: if a test fails I often just copy-paste the command line used as reported by `llvm-lit` quite mechanically to see a broader context than `FileCheck` included in the error message, and it's nice if it happens to produce a shorten output.

I can stop committing these of course.


Repository:
  rL LLVM

https://reviews.llvm.org/D46339





More information about the llvm-commits mailing list