[PATCH] D121272: [CaptureTracking][NFCI] Expose capture tracking logic

Dmitry Sidorov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 14 11:10:46 PDT 2022


sidorovd added inline comments.


================
Comment at: llvm/include/llvm/Analysis/CaptureTracking.h:113
+    MAY_CAPTURE,
+    PASSTHROUGH,
+  };
----------------
Hi, while PASSTHROUGH seems to be a good name for the enum value, it so happened, that it is also a macro defined in wingdi.h (which is included in windows.h). Can we consider renaming it or we should guard/undef the macro in the downstream?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121272



More information about the llvm-commits mailing list