[PATCH] D84551: [GlobalISel] Add G_INTRINSIC_LRINT and translate from llvm.lrint
    Amara Emerson via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Jul 24 14:10:56 PDT 2020
    
    
  
aemerson added a comment.
In D84551#2173159 <https://reviews.llvm.org/D84551#2173159>, @arsenm wrote:
> This will conflict with D84159 <https://reviews.llvm.org/D84159>.
>
> Why does the IR have rint, lrint, and llrint? libm has all 3, but they seem to just differ by the return integer type? Why isn't there just the one rint intrinsic and clang is responsible for picking the right result type?
Conflict in what way?
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84551/new/
https://reviews.llvm.org/D84551
    
    
More information about the llvm-commits
mailing list