[PATCH] D135738: [InstCombine] Bail out of casting calls when a conversion to byval is involved.

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/cast-to-byval.ll:3
+; Check that function calls involving conversion to byval aren't transformed.
+; RUN: opt < %s -passes=instcombine -mtriple=x86_64-unknown-unknown -S | FileCheck %s
+
----------------
glandium wrote:
> nikic wrote:
> > Drop the mtriple here -- that shouldn't be needed and would require a `REQUIRES` to avoid breakage when the target is not built.
> does opt default to 64-bits on 32-bits hosts or does running tests on 32-bits not supported?
Opt uses a 64-bit data layout by default, independent of host.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135738