[PATCH] D69103: Backend for NEC SX-Aurora

Tue Oct 22 17:31:51 PDT 2019

asl added inline comments.

================
Comment at: llvm/test/CodeGen/VE/target_support.ll:1
+; RUN: not not llc < %s -asm-verbose=false -march=ve 2>&1 | FileCheck %s --check-prefix=ARCH
+; RUN: not not llc < %s -asm-verbose=false -mtriple=ve 2>&1 | FileCheck %s --check-prefix=TRIPLE
----------------
simoll wrote:
> bcain wrote:
> > `not not` can't be right, can it?  Just run `llc` directly.
> The 'not not' makes sure the test does not crash when 'llc' crashes (which is anticipated in a successful run). We need this because if the tests succeeds, the 'llc' call crashes trying to create initialize VE codegen. The test fails when 'llc' reports that 've' is not a valid target.
Can't you just check the output of llc --version then? Expecting llc to crash is quite ugly...

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69103