[PATCH] D72260: AMDGPU/GlobalISel: Add pre-legalize combiner pass

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 17 18:01:48 PST 2020


arsenm marked an inline comment as done.
arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUPreLegalizerCombiner.cpp:113
+
+AMDGPUPreLegalizerCombiner::AMDGPUPreLegalizerCombiner(bool IsOptNone)
+    : MachineFunctionPass(ID), IsOptNone(IsOptNone) {
----------------
kerbowa wrote:
> I understand this stub pass is copied from AArch64, but the indentation style below doesn't really match the rest of the AMDGPU GlobalISel patches.
I think this matches what clang-format does


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

https://reviews.llvm.org/D72260





More information about the llvm-commits mailing list