[PATCH] D126889: [NFC] format InstructionSimplify & lowerCaseFunctionNames

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 2 09:37:07 PDT 2022


spatel added a comment.

I should have been clearer with my original suggestion - I was only thinking of changing the static/internal function names within InstSimplify itself, so it would still be a one-file patch.

I realize that doesn't fix the overall formatting mess of LLVM, but it would hopefully make InstructionSimplify.cpp alone a little easier to read.

I don't object to the larger change, but I only work on the main branch. AFAIK, we've avoided doing something this big because it makes life harder for maintainers of private forks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126889



More information about the llvm-commits mailing list