[PATCH] D38043: Restore ability for C++ API users to Enable IPRA.

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 20 03:18:15 PDT 2017


dsanders added inline comments.


================
Comment at: lib/CodeGen/TargetPassConfig.cpp:376
     // If not explicitly specified, use target default.
-    TM.Options.EnableIPRA = TM.useIPRA();
+    TM.Options.EnableIPRA |= TM.useIPRA();
   }
----------------
arsenm wrote:
> I'm confused why you would want to still be directly setting this instead of using the new hook
For a given target, the hook only allows us to enable it for every compile or disable it for every compile. We need to be able to enable it for some compiles and disable it for others.


https://reviews.llvm.org/D38043





More information about the llvm-commits mailing list