[PATCH] D122087: Add HLSL Language Option and Preprocessor

[Erich Keane via Phabricator via cfe-commits]

erichkeane added inline comments.


================
Comment at: clang/lib/Frontend/InitPreprocessor.cpp:401
+    // The current shader stage itself
+    uint32_t StageInteger = StageInteger =
+        (uint32_t)TI.getTriple().getEnvironment() -
----------------
Why does this do a double-assign here?  Is th ere supposed to be a 2nd variable here?  Otherwise this is likely not particularly well-defined behavior.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122087