rsmith added inline comments. ================ Comment at: lib/CodeGen/CodeGenModule.cpp:513 @@ +512,3 @@ +void CodeGenModule::RewriteAlwaysInlineFunctions() { + for (llvm::Function *Fn : AlwaysInlineFunctions) + RewriteAlwaysInlineFunction(Fn); ---------------- Does it matter that this traversal is nondeterministic? (Is the nondeterminism visible in the output IR?) Could you use a vector type for `AlwaysInlineFunctions`? (Does the same function ever actually get added to it twice?) If not, maybe a `SetVector` would be more appropriate.
http://reviews.llvm.org/D12087 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits