[PATCH] D99438: [SimplifyLibCalls] Take size of int into consideration when emitting ldexp/ldexpf
Bjorn Pettersson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri May 21 06:38:41 PDT 2021
bjope marked 4 inline comments as done.
bjope added inline comments.
================
Comment at: llvm/test/Transforms/InstCombine/pow_fp_int.ll:2
; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
-; RUN: opt -instcombine -S < %s | FileCheck %s
+; RUN: opt -mtriple unknown -instcombine -S < %s | FileCheck %s
----------------
xbolva00 wrote:
> Why this change?
The test case now depends on the default triple (there are some tests checking for ldexpf). So if building with `msp430` or `avr` being the default target the test would fail otherwise.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99438/new/
https://reviews.llvm.org/D99438
More information about the llvm-commits
mailing list