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
