[PATCH] D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev marked 5 inline comments as done.
fedor.sergeev added inline comments.


================
Comment at: lib/Transforms/IPO/FunctionAttrs.cpp:1151
+      if (InferInThisFunc.empty())
+        break;
     }
----------------
jlebar wrote:
> If you move this check to the top of the loop, then you don't need the check before the loop.
Perhaps its a bit of over-optimization, but I dont want to touch instructions(F) (enter the instructions loop) just to determine that we need to break out.
I dont feel that having these two checks is detrimental to readability.


Repository:
  rL LLVM

https://reviews.llvm.org/D44415