[PATCH] D43781: Make AddrSpaceCast noops on PPC

Valentin Churavy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 19 09:09:29 PDT 2018


vchuravy marked an inline comment as done.
vchuravy added a comment.

Thanks! Can you land this for me I don't have the commit bit.



================
Comment at: test/CodeGen/PowerPC/addrspacecast.ll:1
+; RUN: llc -mtriple=powerpc64le-unknown-linux-gnu %s -o /dev/null
+
----------------
nemanjai wrote:
> I'm not sure how `/dev/null` works for targets I'm not familiar with (like Windows). Is this a common thing to do?
> It might not be a terrible thing to actually do `< %s | FileCheck %s` and then maybe just a `CHECK-LABEL` to verify we've actually generated assembly.
> 
> But in either case, I assume that `lit` checks for the return code from the `llc` invocation, so I'm fine either way.
Yeah this seems fairly standard and was taken from the `ARM/addrspacecast.ll`
But I can use FileCheck and CHECK-LABEL to do the same


Repository:
  rL LLVM

https://reviews.llvm.org/D43781





More information about the llvm-commits mailing list