[PATCH] D156397: [FunctionAttrs] Unconditionally perform argument attribute inference in the first function-attrs pass

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

last nit then I think this is good to go
`function-attrs<skip-non-recursive>` should be renamed to something like `function-attrs<non-recursive-only-arg-attrs>`



================
Comment at: llvm/lib/Transforms/Utils/SimplifyCFG.cpp:7340
 bool SimplifyCFGOpt::run(BasicBlock *BB) {
+  return false;
   bool Changed = false;
----------------
nikic wrote:
> Debugging leftover?
unintenitonal change?


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

https://reviews.llvm.org/D156397