beanz 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() -
----------------
erichkeane wrote:
> 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.
I must have screwed this up in rebating one of the patches. Will fix ASAP.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122087

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

Reply via email to