[PATCH] D29306: Expose isLegalToPromot as a global helper function so that SamplePGO pass can call it for legality check.

[David Li via Phabricator via llvm-commits]

davidxl added inline comments.


================
Comment at: lib/Transforms/Instrumentation/IndirectCallPromotion.cpp:149
     OK,                   // Should be able to promote.
     NotAvailableInModule, // Cannot find the target in current module.
   };
----------------
Why keeping this one value but eliminating all the others?


================
Comment at: lib/Transforms/Instrumentation/IndirectCallPromotion.cpp:154
+  bool isPromotionLegal(Instruction *Inst, uint64_t Target, Function *&F,
+                        const char **Reason=nullptr);
 
----------------
missing space


https://reviews.llvm.org/D29306