[PATCH] D110288: Move pow transformations to sqrt/cbrt to earlier in the compiler pipeline

Masoud Ataei via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 6 14:47:07 PDT 2021


masoud.ataei marked 2 inline comments as done.
masoud.ataei added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/pow.75.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt -O3 -S -mtriple=aarch64-- %s 2>&1 | FileCheck %s
 ; REQUIRES: asserts
----------------
bmahjour wrote:
> bmahjour wrote:
> > Is -O3 really the minimum opt level that would trigger this transformation?
> Also, these lit tests no longer test codegen, so they should probably be moved elsewhere... under Transforms subdirectories.
-O2 is minimum.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110288



More information about the llvm-commits mailing list