[PATCH] D36195: Fix the bug that parseAAPipeline is not invoked in runNewPMPasses in release compiler.

[Dehao Chen via Phabricator via llvm-commits]

danielcdh created this revision.
Herald added subscribers: mehdi_amini, sanjoy.

The logic is guarded by "assert".


https://reviews.llvm.org/D36195

Files:
  lib/LTO/LTOBackend.cpp


Index: lib/LTO/LTOBackend.cpp
===================================================================
--- lib/LTO/LTOBackend.cpp
+++ lib/LTO/LTOBackend.cpp
@@ -137,7 +137,8 @@
   AAManager AA;
 
   // Parse a custom AA pipeline if asked to.
-  assert(PB.parseAAPipeline(AA, "default"));
+  if (!PB.parseAAPipeline(AA, "default"))
+    llvm_unreachable("Error parsing AA pipeline");
 
   LoopAnalysisManager LAM;
   FunctionAnalysisManager FAM;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36195.109234.patch
Type: text/x-patch
Size: 440 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170801/14145217/attachment.bin>