[PATCH] D78662: [builtins] Support architectures with 16-bit int

[Mikael Holmén via Phabricator via llvm-commits]

uabelho added inline comments.


================
Comment at: compiler-rt/lib/builtins/udivmoddi4.c:85
       }
       return n.s.high >> __builtin_ctz(d.s.high);
     }
----------------
Did you consider changing this call into the new macro ctzsi?

Our downstream target also has ints that are 16 bits wide and we've had some downstream changes somewhat similar to this patch, but then we also changed this call (and another one to __builtin_ctz in this file) to __builtin_ctzl.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78662