saiislam marked 3 inline comments as done.
saiislam added inline comments.

================
Comment at: clang/tools/clang-offload-bundler/ClangOffloadBundler.cpp:1048-1049
+    while (!CurKindTriple.empty()) {
+      if (hasHostKind(CurKindTriple)) {
+        // Do nothing, we don't extract host code yet
+      } else if (checkDeviceOptions(getDevice(getTriple(CurKindTriple)),
----------------
ABataev wrote:
> Just `continue` and make `else if` just `if`
wasn't possible with the code flow. there is stuff to be processed in case of 
failure as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93525

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to