[PATCH] D81285: [builtins] Change si_int to int in some helper declarations

Ayke via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 19 05:21:18 PDT 2020


aykevl added a comment.

In D81285#2102948 <https://reviews.llvm.org/D81285#2102948>, @atrosinenko wrote:

> On renaming fixed width integer types to their traditional names: I would prefer sending such patch afterwards, it would probably be as simple as just running `sed --in-place` several times.


Yes definitely. It seems best to me to separate refactorings from functional changes. If it should indeed be done, I'm not a maintainer of compiler-rt.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81285





More information about the cfe-commits mailing list