jdoerfert marked an inline comment as done. jdoerfert added inline comments.
================ Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:394 + + indicateFixpoint(/* Optimistic */ true); + return; ---------------- xbolva00 wrote: > Maybe enum here ? > So you could call indicateFixpoint(Fixpoint::optimistic) ? > > Or maybe even better, indicateOptimisticFixpoint()? Good points. I'll probably go with the second, most explicit, solution. I'll update (all) the source files soon. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59919/new/ https://reviews.llvm.org/D59919 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
