[PATCH] D52854: Use is.constant intrinsic for __builtin_constant_p

Bill Wendling via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 4 13:25:56 PDT 2018


void added inline comments.


================
Comment at: lib/Sema/SemaExpr.cpp:15621-15623
+        for (InitListExpr::iterator II = E->begin(), IE = E->end();
+             II != IE; ++II)
+          Visit(*II);
----------------
nickdesaulniers wrote:
> nickdesaulniers wrote:
> > `std::for_each`?
> Sorry, posted that comment as you uploaded the next version.  This should be highlighting L15618 to L15620.
That's no used anywhere else in the source code. Maybe there's another mechanism that they use?


https://reviews.llvm.org/D52854





More information about the cfe-commits mailing list