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

Mike Hommey via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 13:43:45 PDT 2022


glandium 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
+
----------------
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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135738



More information about the llvm-commits mailing list