[PATCH] D34921: [ConstantHoisting] Remove dupliate logic in constant hoisting

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 9 06:14:15 PDT 2017


efriedma added inline comments.


================
Comment at: lib/Transforms/Utils/Local.cpp:2182
+      }
+    }
     // Many arithmetic intrinsics have no issue taking a
----------------
You can just write "if (isa<InlineAsm>(ImmutableCallSite(I).getCalledValue()))" to handle both Call and Invoke.


https://reviews.llvm.org/D34921





More information about the llvm-commits mailing list