[PATCH] D56491: treat invoke like call

Wei Mi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 15 09:48:13 PST 2019


wmi accepted this revision.
wmi added a comment.
This revision is now accepted and ready to land.

LGTM.



================
Comment at: lib/Transforms/Utils/AddDiscriminators.cpp:239-244
+      if (isa<CallInst>(I)) {
+        if (isa<IntrinsicInst>(I))
+          continue;
+      }
+      else if(!isa<InvokeInst>(I))
         continue;
----------------
nit: Can we merge it to?
```
if ((!isa<CallInst>(I) && !isa<InvokeInst>(I)) || isa<IntrinsicInst>(I))
  continue;
```


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56491/new/

https://reviews.llvm.org/D56491





More information about the llvm-commits mailing list