[PATCH] D55294: [GISel]: Refactor MachineIRBuilder to allow passing additional parameters to build Instrs

Justin Bogner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 10 11:20:07 PST 2018


bogner added a comment.

In D55294#1319186 <https://reviews.llvm.org/D55294#1319186>, @aditya_nandakumar wrote:

> I also added{F7649492 <https://reviews.llvm.org/F7649492>}
>
> F7649491: Build_instr_updateCheck.cpp <https://reviews.llvm.org/F7649491>
>  which is a clang-tidy implementation to automatically add {} to most buildInstr calls.


Please link to this in the commit message itself so it's easier to find once the change is committed. Also, it would be nice to add some comments in the file on how to build and run it with clang-tidy (even if it's just linking to clang-tidy docs).


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55294





More information about the llvm-commits mailing list