arphaman marked an inline comment as done.
arphaman added inline comments.
================
Comment at: clang/lib/Lex/DependencyDirectivesSourceMinimizer.cpp:270
+    return false;
+  if (Prev == '8' && (Cur - 1 != Start) && *(Cur - 2) == 'u')
+    return false;
----------------
Bigcheese wrote:
> Are we sure at this point that it's always safe to jump back 2?
It should be, because otherwise `Start` would've been equals to `Cur - 1` which 
we check for right before the dereference.


Repository:
  rC Clang

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

https://reviews.llvm.org/D64525



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

Reply via email to