Skip to content

Commit e6d4584

Browse files
author
Jan Voung
committed
Put llvm.powi in the dev list of intrinsics.
This is used by the LLVM translator as part of: lib/Analysis/ConstantFolding.cpp (it tries to do constant folding for pow calls...) Also, tweak comment about llvm.pow vs llvm.powi. It looks like powi is the imprecise one, not pow. BUG=unblock DEPs roll, broken self-build. BUG=http://code.google.com/p/nativeclient/issues/detail?id=3378 [email protected] Review URL: https://codereview.chromium.org/14631013
1 parent 8c9803a commit e6d4584

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

lib/Analysis/NaCl/PNaClABIVerifyModule.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,8 @@ bool PNaClABIVerifyModule::IsWhitelistedIntrinsic(const Function* F,
174174
case Intrinsic::log2: // Rounding not defined: support with fast-math?
175175
case Intrinsic::log10: // Rounding not defined: support with fast-math?
176176
case Intrinsic::nacl_target_arch: // Used by translator self-build.
177-
case Intrinsic::pow: // Rounding not defined: support with fast-math?
177+
case Intrinsic::pow: // Rounding is supposed to be the same as libm.
178+
case Intrinsic::powi: // Rounding not defined: support with fast-math?
178179
case Intrinsic::prefetch: // Could ignore if target doesn't support?
179180
case Intrinsic::returnaddress: // Support for 0-level or not?
180181
case Intrinsic::sin: // Rounding not defined: support with fast-math?

0 commit comments

Comments
 (0)