[PATCH] D53053: [Intrinsic] Signed Saturation Addition Intrinsic

Leonard Chan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 15 10:05:16 PDT 2018


leonardchan added inline comments.


================
Comment at: test/CodeGen/X86/sadd_sat.ll:2
+; RUN: llc < %s -mcpu=generic -mtriple=x86_64-linux | FileCheck %s
+; RUN: llc < %s -mcpu=generic -march=x86 | FileCheck %s --check-prefix=CHECK32
+
----------------
craig.topper wrote:
> craig.topper wrote:
> > Use -mtriple=i686-- not -march. And generate the checks using utils/update_llc_test_checks.py
> It might also be better to add -mattr=cmov on the 32-bit test
This is a very nifty script!


Repository:
  rL LLVM

https://reviews.llvm.org/D53053





More information about the llvm-commits mailing list