[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