[PATCH] D37877: Update getMergedLocation to check the instruction type and merge properly.

Dehao Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 19:02:19 PDT 2017


danielcdh marked 2 inline comments as done.
danielcdh added inline comments.


================
Comment at: include/llvm/IR/DebugInfoMetadata.h:1447
 
+  static void applyMergedLocation(
+      Instruction *Inst, const DILocation *LocA, const DILocation *LocB);
----------------
aprantl wrote:
> Sorry, one more thing: Why is this a static function instead of a member?
Moved the API to Instruction::applyMergedLocation.


https://reviews.llvm.org/D37877





More information about the llvm-commits mailing list