[PATCH] D41832: LLParser: Do not check alloca addr space if the assembly does not contain data layout definition

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 8 13:30:18 PST 2018


arsenm added inline comments.


================
Comment at: include/llvm/IR/Verifier.h:101
+                  bool *BrokenDebugInfo = nullptr,
+                  bool CheckAllocaAddrSpace = true);
 
----------------
I'm not sure I understand why this option is needed or to UpgradeDebugInfo. I would expect the datalayout to be set already by the time any verification is done


https://reviews.llvm.org/D41832





More information about the llvm-commits mailing list