chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.
LGTM with a tiny tweak below.
Would be good to add a test that this flag is being honored, either in this
patch or in a follow-up.
================
Comment at: lib/CodeGen/BackendUtil.cpp:862-863
+ PGOOpt = PGOOptions("", "", "", false, true);
+ else
+ PGOOpt = None;
----------------
No need for this? The optional should be disengaged by default when constructed
as you do above.
https://reviews.llvm.org/D35746
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
